1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[ruby/yarp] Clang beats gcc in pedantry

Browse Source 
Accidentally declared int32_t and not uint32_t and it was caught
on macos.

https://github.com/ruby/yarp/commit/6581520793
This commit is contained in:
Thomas E. Enebo 2023-08-03 14:19:49 -04:00 committed by Takashi Kokubun
parent 1e0749e032
commit 7257145320
Notes: git 2023-08-17 00:48:05 +00:00 
Merged: https://github.com/ruby/ruby/pull/8226
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
yarp/yarp.c
View File

@ -12924,7 +12924,7 @@ yp_process_metadata(yp_parser_t *parser, const char *metadata) {
yp_parser_scope_push(parser, scope_index == 0);
for (size_t variable_index = 0; variable_index < number_of_variables; variable_index++) {
int32_t length = (uint32_t) *p;
uint32_t length = (uint32_t) *p;
p += 4;
yp_parser_local_add_location(parser, p, p + length);