bzip2: remove redundant clearing of an alredy unset bit
authorDenys Vlasenko <vda.linux@googlemail.com>
Sat, 3 Feb 2018 16:38:23 +0000 (17:38 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sat, 3 Feb 2018 16:38:23 +0000 (17:38 +0100)
function                                             old     new   delta
mainSort                                            1178    1171      -7

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
archival/libarchive/bz/blocksort.c

index 3fe74f71cb3155362a990f5a163c5fed8a3e2455..473244db01a49a26f7938ed0f32d9f2ef38bbb21 100644 (file)
@@ -873,10 +873,10 @@ void mainSort(EState* state,
                 */
                for (j = 0; j <= 255; j++) {
                        if (j != ss) {
-                               int32_t sb;
+                               unsigned sb;
                                sb = (ss << 8) + j;
                                if (!(ftab[sb] & SETMASK)) {
-                                       int32_t lo =  ftab[sb]   & CLEARMASK;
+                                       int32_t lo =  ftab[sb] /*& CLEARMASK (redundant)*/;
                                        int32_t hi = (ftab[sb+1] & CLEARMASK) - 1;
                                        if (hi > lo) {
                                                mainQSort3(