# HG changeset patch # User Christian Brabandt # Date 1706212805 -3600 # Node ID 5aee5855ccfe88bc997f1ed0e0c95c4a24a2136d # Parent 698542032507952c7b03b4d73d3158c41d0c6785 patch 9.1.0052: Patch 9.1.0041 causes regressions for users Commit: https://github.com/vim/vim/commit/fa8c9715587a24562dd755c44dc13aef43d7d875 Author: Christian Brabandt Date: Thu Jan 25 20:50:49 2024 +0100 patch 9.1.0052: Patch 9.1.0041 causes regressions for users Problem: Patch 9.1.0041 causes regressions for users (Gleb Fotengauer-Malinovskiy) Solution: Revert 9.1.0041 and restore old behaviour Revert "patch 9.1.0041: xxd -i may generate incorrect C statements" This reverts commit 7062be13129985fe297b9a8e59c57b8f0db61b8f. Signed-off-by: Christian Brabandt diff --git a/src/testdir/test_xxd.vim b/src/testdir/test_xxd.vim --- a/src/testdir/test_xxd.vim +++ b/src/testdir/test_xxd.vim @@ -103,7 +103,7 @@ func Test_xxd() unsigned char XXDfile[] = { 0x54, 0x45, 0x53, 0x54, 0x61, 0x62, 0x63, 0x64, 0x30, 0x39, 0x0a }; - size_t XXDfile_len = 11; + unsigned int XXDfile_len = 11; [CODE] call assert_equal(expected, getline(1,'$'), s:Mess(s:test)) @@ -119,7 +119,7 @@ func Test_xxd() unsigned char XXDFILE[] = { 0x54, 0x45, 0x53, 0x54, 0x61, 0x62, 0x63, 0x64, 0x30, 0x39, 0x0a }; - size_t XXDFILE_LEN = 11; + unsigned int XXDFILE_LEN = 11; [CODE] call assert_equal(expected, getline(1,'$'), s:Mess(s:test)) endfor @@ -233,7 +233,7 @@ func Test_xxd() unsigned char varName[] = { 0x54, 0x45, 0x53, 0x54, 0x61, 0x62, 0x63, 0x64, 0x30, 0x39, 0x0a }; - size_t varName_len = 11; + unsigned int varName_len = 11; [CODE] call assert_equal(expected, getline(1,'$'), s:Mess(s:test)) @@ -247,7 +247,7 @@ func Test_xxd() unsigned char StdIn[] = { 0x54, 0x45, 0x53, 0x54, 0x61, 0x62, 0x63, 0x64, 0x30, 0x39, 0x0a }; - size_t StdIn_len = 11; + unsigned int StdIn_len = 11; [CODE] call assert_equal(expected, getline(1,'$'), s:Mess(s:test)) @@ -263,7 +263,7 @@ func Test_xxd() unsigned char VARNAME[] = { 0x54, 0x45, 0x53, 0x54, 0x61, 0x62, 0x63, 0x64, 0x30, 0x39, 0x0a }; - size_t VARNAME_LEN = 11; + unsigned int VARNAME_LEN = 11; [CODE] call assert_equal(expected, getline(1,'$'), s:Mess(s:test)) endfor diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -705,6 +705,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 52, +/**/ 51, /**/ 50, diff --git a/src/xxd/xxd.c b/src/xxd/xxd.c --- a/src/xxd/xxd.c +++ b/src/xxd/xxd.c @@ -60,6 +60,7 @@ * 06.10.2023 enable -r -b to reverse bit dumps * 12.01.2024 disable auto-conversion for z/OS (MVS) * 17.01.2024 use size_t instead of usigned int for code-generation (-i), #13876 + * 25.01.2024 revert the previous patch (size_t instead of unsigned int) * * (c) 1990-1998 by Juergen Weigert (jnweiger@gmail.com) * @@ -140,7 +141,7 @@ extern void perror __P((char *)); # endif #endif -char version[] = "xxd 2024-01-17 by Juergen Weigert et al."; +char version[] = "xxd 2024-01-25 by Juergen Weigert et al."; #ifdef WIN32 char osver[] = " (Win32)"; #else @@ -978,7 +979,7 @@ main(int argc, char *argv[]) if (varname != NULL) { fputs_or_die("};\n", fpo); - FPRINTF_OR_DIE((fpo, "size_t %s", isdigit((unsigned char)varname[0]) ? "__" : "")); + FPRINTF_OR_DIE((fpo, "unsigned int %s", isdigit((unsigned char)varname[0]) ? "__" : "")); for (e = 0; (c = varname[e]) != 0; e++) putc_or_die(isalnum((unsigned char)c) ? CONDITIONAL_CAPITALIZE(c) : '_', fpo); FPRINTF_OR_DIE((fpo, "_%s = %d;\n", capitalize ? "LEN" : "len", p));