{"thread":{"id":"46646","subject":"mk-dontmerge/size-t-on-next test failure","startedAt":"2017-08-22T17:06:32Z","lastAt":"2017-08-22T19:24:40Z","messageCount":2,"participants":["Johannes Sixt","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"326944","messageId":"fcddd218-e104-0f96-71e6-4522564555f1@kdbg.org","threadId":"46646","inReplyTo":null,"subject":"mk-dontmerge/size-t-on-next test failure","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2017-08-22T17:06:24Z","receivedAt":"2017-08-22T17:06:32Z","isPatch":false,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"I observe the test failure below in t0040-parse-options.sh. It bisects\nto 1a7909b25eb4ab3071ce4290115618e2582eadaa \"Convert pack-objects to\nsize_t\". It looks like git_parse_size_t() needs a fix. This is on\nWindows, 32 bit. size_t, int and long are all 32 bits wide.\n\nexpecting success:\n        check magnitude: 1073741824 -m 1g\n\nok 18 - OPT_MAGNITUDE() giga\n\nexpecting success:\n        check magnitude: 3221225472 -m 3g\n\nerror: switch `m' expects a non-negative integer value with an optional\nk/m/g suffix\nusage: test-parse-options <options>\n\n    --yes                 get a boolean\n    -D, --no-doubt        begins with 'no-'\n    -B, --no-fear         be brave\n    -b, --boolean         increment by one\n    -4, --or4             bitwise-or boolean with ...0100\n    --neg-or4             same as --no-or4\n\n    -i, --integer <n>     get a integer\n    -j <n>                get a integer, too\n    -m, --magnitude <n>   get a magnitude\n    --set23               set integer to 23\n    -t <time>             get timestamp of <time>\n    -L, --length <str>    get length of <str>\n    -F, --file <file>     set file to <file>\n\nString options\n    -s, --string <string>\n                          get a string\n    --string2 <str>       get another string\n    --st <st>             get another string (pervert ordering)\n    -o <str>              get another string\n    --list <str>          add str to list\n\nMagic arguments\n    --quux                means --quux\n    -NUM                  set integer to NUM\n    +                     same as -b\n    --ambiguous           positive ambiguity\n    --no-ambiguous        negative ambiguity\n\nStandard options\n    --abbrev[=<n>]        use <n> digits to display SHA-1s\n    -v, --verbose         be verbose\n    -n, --dry-run         dry run\n    -q, --quiet           be quiet\n    --expect <string>     expected output in the variable dump\n\nnot ok 19 - OPT_MAGNITUDE() 3giga\n#\n#               check magnitude: 3221225472 -m 3g\n#\n"},{"id":"326957","messageId":"xmqqk21v2yof.fsf@gitster.mtv.corp.google.com","threadId":"46646","inReplyTo":"fcddd218-e104-0f96-71e6-4522564555f1@kdbg.org","subject":"Re: mk-dontmerge/size-t-on-next test failure","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-08-22T19:24:32Z","receivedAt":"2017-08-22T19:24:40Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Sixt <j6t@kdbg.org> writes:\n\n> I observe the test failure below in t0040-parse-options.sh. It bisects\n> to 1a7909b25eb4ab3071ce4290115618e2582eadaa \"Convert pack-objects to\n> size_t\". It looks like git_parse_size_t() needs a fix. This is on\n> Windows, 32 bit. size_t, int and long are all 32 bits wide.\n\nThanks.  It is very much appreciated for people to peek at the\nbranch and try on different platforms that I (or Martin) do not use\nregularly.\n\n"}]}