Skip to content

Commit 47e4090

Browse files
committed
fix function signatures for parity with native functions in PHP 8
1 parent c0fee7a commit 47e4090

File tree

2 files changed

+11
-12
lines changed

2 files changed

+11
-12
lines changed

src/Iconv/bootstrap80.php

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,19 +25,19 @@
2525
}
2626

2727
if (!function_exists('iconv')) {
28-
function iconv(?string $from_encoding, ?string $to_encoding, ?string $string): string|false { return p\Iconv::iconv((string) $from_encoding, (string) $to_encoding, (string) $string); }
28+
function iconv(string $from_encoding, string $to_encoding, string $string): string|false { return p\Iconv::iconv((string) $from_encoding, (string) $to_encoding, (string) $string); }
2929
}
3030
if (!function_exists('iconv_get_encoding')) {
31-
function iconv_get_encoding(?string $type = 'all'): array|string|false { return p\Iconv::iconv_get_encoding((string) $type); }
31+
function iconv_get_encoding(string $type = 'all'): array|string|false { return p\Iconv::iconv_get_encoding((string) $type); }
3232
}
3333
if (!function_exists('iconv_set_encoding')) {
34-
function iconv_set_encoding(?string $type, ?string $encoding): bool { return p\Iconv::iconv_set_encoding((string) $type, (string) $encoding); }
34+
function iconv_set_encoding(string $type, string $encoding): bool { return p\Iconv::iconv_set_encoding((string) $type, (string) $encoding); }
3535
}
3636
if (!function_exists('iconv_mime_encode')) {
37-
function iconv_mime_encode(?string $field_name, ?string $field_value, ?array $options = []): string|false { return p\Iconv::iconv_mime_encode((string) $field_name, (string) $field_value, (array) $options); }
37+
function iconv_mime_encode(string $field_name, string $field_value, array $options = []): string|false { return p\Iconv::iconv_mime_encode((string) $field_name, (string) $field_value, (array) $options); }
3838
}
3939
if (!function_exists('iconv_mime_decode_headers')) {
40-
function iconv_mime_decode_headers(?string $headers, ?int $mode = 0, ?string $encoding = null): array|false { return p\Iconv::iconv_mime_decode_headers((string) $headers, (int) $mode, $encoding); }
40+
function iconv_mime_decode_headers(string $headers, int $mode = 0, ?string $encoding = null): array|false { return p\Iconv::iconv_mime_decode_headers((string) $headers, (int) $mode, $encoding); }
4141
}
4242

4343
if (extension_loaded('mbstring')) {
@@ -59,22 +59,22 @@ function iconv_mime_decode($string, $mode = 0, $encoding = null) { $currentMbEnc
5959
} else {
6060
if (!function_exists('iconv_strlen')) {
6161
if (extension_loaded('xml')) {
62-
function iconv_strlen(?string $string, ?string $encoding = null): int|false { return p\Iconv::strlen1((string) $string, $encoding); }
62+
function iconv_strlen(string $string, ?string $encoding = null): int|false { return p\Iconv::strlen1((string) $string, $encoding); }
6363
} else {
64-
function iconv_strlen(?string $string, ?string $encoding = null): int|false { return p\Iconv::strlen2((string) $string, $encoding); }
64+
function iconv_strlen(string $string, ?string $encoding = null): int|false { return p\Iconv::strlen2((string) $string, $encoding); }
6565
}
6666
}
6767

6868
if (!function_exists('iconv_strpos')) {
69-
function iconv_strpos(?string $haystack, ?string $needle, ?int $offset = 0, ?string $encoding = null): int|false { return p\Iconv::iconv_strpos((string) $haystack, (string) $needle, (int) $offset, $encoding); }
69+
function iconv_strpos(string $haystack, string $needle, int $offset = 0, ?string $encoding = null): int|false { return p\Iconv::iconv_strpos((string) $haystack, (string) $needle, (int) $offset, $encoding); }
7070
}
7171
if (!function_exists('iconv_strrpos')) {
72-
function iconv_strrpos(?string $haystack, ?string $needle, ?string $encoding = null): int|false { return p\Iconv::iconv_strrpos((string) $haystack, (string) $needle, $encoding); }
72+
function iconv_strrpos(string $haystack, string $needle, ?string $encoding = null): int|false { return p\Iconv::iconv_strrpos((string) $haystack, (string) $needle, $encoding); }
7373
}
7474
if (!function_exists('iconv_substr')) {
75-
function iconv_substr(?string $string, ?int $offset, ?int $length = null, ?string $encoding = null): string|false { return p\Iconv::iconv_substr((string) $string, (string) $offset, $length, $encoding); }
75+
function iconv_substr(string $string, int $offset, ?int $length = null, ?string $encoding = null): string|false { return p\Iconv::iconv_substr((string) $string, (string) $offset, $length, $encoding); }
7676
}
7777
if (!function_exists('iconv_mime_decode')) {
78-
function iconv_mime_decode(?string $string, ?int $mode = 0, ?string $encoding = null): string|false { return p\Iconv::iconv_mime_decode((string) $string, (int) $mode, $encoding); }
78+
function iconv_mime_decode(string $string, int $mode = 0, ?string $encoding = null): string|false { return p\Iconv::iconv_mime_decode((string) $string, (int) $mode, $encoding); }
7979
}
8080
}

src/Util/TestListenerTrait.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,6 @@ function {$f['name']}{$f['signature']}
124124
$polyfillSignature = ReflectionCaster::getSignature($polyfillSignature);
125125

126126
$map = [
127-
'?' => '',
128127
'IDNA_DEFAULT' => \PHP_VERSION_ID >= 80100 ? 'IDNA_DEFAULT' : '0',
129128
'array|string|null $string' => 'array|string $string',
130129
'array|string|null $from_encoding = null' => 'array|string|null $from_encoding = null',

0 commit comments

Comments
 (0)