Repository navigation
vlib: change byte types to u8 types - #19930
Merged
Merged
Conversation
Member
|
Thanks! :) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I went looking through the files under vlib to see if any of them still used the byte type and replaced likely candidates with u8. I don't think I was too aggressive considering that the byte type still works but I thought it would be a good idea to go ahead and change these cases.
π€[deprecated] Generated by Copilot at ac46001
This pull request replaces the use of
byteand&bytetypes withu8and&u8types in various modules and tests, following the recent renaming of thebytealias tou8in the V language. This improves consistency, clarity, and compatibility across the codebase.π€[deprecated] Generated by Copilot at ac46001
&byteor[]byteto&u8or[]u8to comply with the new aliasing rules in V (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)linuxsys_bare.vfile tomemory_managment.vand update the functions in it (link, link)bytetype name from the tests and useu8instead (link, link, link, link, link, link, link)stringstruct and thetos,tos2, andi64_tosfunctions instring_bare.v(link, link, link, link)wyhashfunction inmap.c.v(link)vsprintf,vsnprintf,bare_read,bare_print,bare_eprint, andwritefunctions inwasm_bare/libc_impl.c.v(link, link)arc4random_buffunction inrand_freebsd.c.vandrand_openbsd.c.v(link, link)getrandomfunction inrand_solaris.c.v(link)strip_newlinesfunction inbase32.v(link)stbi_write_hdrfunction instbi.c.v(link, link)test_assign_array_of_aliasesfunction inchecker/assign.v(link)get_bytes_arrayandtest_aliased_array_method_callfunctions intests/aliased_array_method_call_test.v(link, link)test_assign_array_of_aliasesfunction intests/assign_array_of_aliases_test.v(link)check_init_value_for_arrays_of_option[byte]()line from thetest_check_init_value_for_arrays_of_optionfunction intests/check_init_value_for_arrays_of_option_test.v(link)UUIDtype and thecodec_test_uuidconstant intests/const_from_bytes_test.v(link)f1andf2functions intests/fn_call_mut_array_of_aliases_args_test.v(link)test_fn_with_array_of_aliases_argumentfunction intests/fn_with_array_of_aliases_argument_test.v(link)assert typeof(byte(1)).name == 'byte'line from thetest_typeof_simple_typesfunction intests/typeof_simple_types_test.v(link)