Diff of the two buildlogs: -- --- b1/build.log 2024-04-27 09:40:18.465327901 +0000 +++ b2/build.log 2024-04-27 09:40:54.259800508 +0000 @@ -1,6 +1,6 @@ I: pbuilder: network access will be disabled during build -I: Current time: Fri May 30 04:02:50 -12 2025 -I: pbuilder-time-stamp: 1748620970 +I: Current time: Sat Apr 27 23:40:21 +14 2024 +I: pbuilder-time-stamp: 1714210821 I: Building the build Environment I: extracting base tarball [/var/cache/pbuilder/trixie-reproducible-base.tgz] I: copying local configuration @@ -31,52 +31,84 @@ dpkg-source: info: applying cross-build.patch I: Not using root during the build. I: Installing the build-deps -I: user script /srv/workspace/pbuilder/1496476/tmp/hooks/D02_print_environment starting +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/D01_modify_environment starting +debug: Running on codethink04-arm64. +I: Changing host+domainname to test build reproducibility +I: Adding a custom variable just for the fun of it... +I: Changing /bin/sh to bash +'/bin/sh' -> '/bin/bash' +lrwxrwxrwx 1 root root 9 Apr 27 09:40 /bin/sh -> /bin/bash +I: Setting pbuilder2's login shell to /bin/bash +I: Setting pbuilder2's GECOS to second user,second room,second work-phone,second home-phone,second other +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/D01_modify_environment finished +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/D02_print_environment starting I: set - BUILDDIR='/build/reproducible-path' - BUILDUSERGECOS='first user,first room,first work-phone,first home-phone,first other' - BUILDUSERNAME='pbuilder1' - BUILD_ARCH='arm64' - DEBIAN_FRONTEND='noninteractive' + BASH=/bin/sh + BASHOPTS=checkwinsize:cmdhist:complete_fullquote:extquote:force_fignore:globasciiranges:globskipdots:hostcomplete:interactive_comments:patsub_replacement:progcomp:promptvars:sourcepath + BASH_ALIASES=() + BASH_ARGC=() + BASH_ARGV=() + BASH_CMDS=() + BASH_LINENO=([0]="12" [1]="0") + BASH_LOADABLES_PATH=/usr/local/lib/bash:/usr/lib/bash:/opt/local/lib/bash:/usr/pkg/lib/bash:/opt/pkg/lib/bash:. + BASH_SOURCE=([0]="/tmp/hooks/D02_print_environment" [1]="/tmp/hooks/D02_print_environment") + BASH_VERSINFO=([0]="5" [1]="2" [2]="21" [3]="1" [4]="release" [5]="aarch64-unknown-linux-gnu") + BASH_VERSION='5.2.21(1)-release' + BUILDDIR=/build/reproducible-path + BUILDUSERGECOS='second user,second room,second work-phone,second home-phone,second other' + BUILDUSERNAME=pbuilder2 + BUILD_ARCH=arm64 + DEBIAN_FRONTEND=noninteractive DEB_BUILD_OPTIONS='buildinfo=+all reproducible=+all parallel=12 ' - DISTRIBUTION='trixie' - HOME='/root' - HOST_ARCH='arm64' + DIRSTACK=() + DISTRIBUTION=trixie + EUID=0 + FUNCNAME=([0]="Echo" [1]="main") + GROUPS=() + HOME=/root + HOSTNAME=i-capture-the-hostname + HOSTTYPE=aarch64 + HOST_ARCH=arm64 IFS=' ' - INVOCATION_ID='aec9d2172d72410da564dd8b3679d0dc' - LANG='C' - LANGUAGE='en_US:en' - LC_ALL='C' - MAIL='/var/mail/root' - OPTIND='1' - PATH='/usr/sbin:/usr/bin:/sbin:/bin:/usr/games' - PBCURRENTCOMMANDLINEOPERATION='build' - PBUILDER_OPERATION='build' - PBUILDER_PKGDATADIR='/usr/share/pbuilder' - PBUILDER_PKGLIBDIR='/usr/lib/pbuilder' - PBUILDER_SYSCONFDIR='/etc' - PPID='1496476' - PS1='# ' - PS2='> ' + INVOCATION_ID=8518aff56631439f8a6ca0a10a458680 + LANG=C + LANGUAGE=nl_BE:nl + LC_ALL=C + MACHTYPE=aarch64-unknown-linux-gnu + MAIL=/var/mail/root + OPTERR=1 + OPTIND=1 + OSTYPE=linux-gnu + PATH=/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/i/capture/the/path + PBCURRENTCOMMANDLINEOPERATION=build + PBUILDER_OPERATION=build + PBUILDER_PKGDATADIR=/usr/share/pbuilder + PBUILDER_PKGLIBDIR=/usr/lib/pbuilder + PBUILDER_SYSCONFDIR=/etc + PIPESTATUS=([0]="0") + POSIXLY_CORRECT=y + PPID=1650357 PS4='+ ' - PWD='/' - SHELL='/bin/bash' - SHLVL='2' - SUDO_COMMAND='/usr/bin/timeout -k 18.1h 18h /usr/bin/ionice -c 3 /usr/bin/nice /usr/sbin/pbuilder --build --configfile /srv/reproducible-results/rbuild-debian/r-b-build.8zDIeDjJ/pbuilderrc_7sai --distribution trixie --hookdir /etc/pbuilder/first-build-hooks --debbuildopts -b --basetgz /var/cache/pbuilder/trixie-reproducible-base.tgz --buildresult /srv/reproducible-results/rbuild-debian/r-b-build.8zDIeDjJ/b1 --logfile b1/build.log wham-align_0.1.5-8.dsc' - SUDO_GID='109' - SUDO_UID='104' - SUDO_USER='jenkins' - TERM='unknown' - TZ='/usr/share/zoneinfo/Etc/GMT+12' - USER='root' - _='/usr/bin/systemd-run' - http_proxy='http://192.168.101.4:3128' + PWD=/ + SHELL=/bin/bash + SHELLOPTS=braceexpand:errexit:hashall:interactive-comments:posix + SHLVL=3 + SUDO_COMMAND='/usr/bin/timeout -k 24.1h 24h /usr/bin/ionice -c 3 /usr/bin/nice -n 11 /usr/bin/unshare --uts -- /usr/sbin/pbuilder --build --configfile /srv/reproducible-results/rbuild-debian/r-b-build.8zDIeDjJ/pbuilderrc_SZFU --distribution trixie --hookdir /etc/pbuilder/rebuild-hooks --debbuildopts -b --basetgz /var/cache/pbuilder/trixie-reproducible-base.tgz --buildresult /srv/reproducible-results/rbuild-debian/r-b-build.8zDIeDjJ/b2 --logfile b2/build.log wham-align_0.1.5-8.dsc' + SUDO_GID=109 + SUDO_UID=104 + SUDO_USER=jenkins + TERM=unknown + TZ=/usr/share/zoneinfo/Etc/GMT-14 + UID=0 + USER=root + _='I: set' + http_proxy=http://192.168.101.4:3128 I: uname -a - Linux codethink03-arm64 6.1.0-20-cloud-arm64 #1 SMP Debian 6.1.85-1 (2024-04-11) aarch64 GNU/Linux + Linux i-capture-the-hostname 6.1.0-20-cloud-arm64 #1 SMP Debian 6.1.85-1 (2024-04-11) aarch64 GNU/Linux I: ls -l /bin - lrwxrwxrwx 1 root root 7 May 24 13:38 /bin -> usr/bin -I: user script /srv/workspace/pbuilder/1496476/tmp/hooks/D02_print_environment finished + lrwxrwxrwx 1 root root 7 Apr 23 11:23 /bin -> usr/bin +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/D02_print_environment finished -> Attempting to satisfy build-dependencies -> Creating pbuilder-satisfydepends-dummy package Package: pbuilder-satisfydepends-dummy @@ -142,7 +174,7 @@ Get: 28 http://deb.debian.org/debian trixie/main arm64 intltool-debian all 0.35.0+20060710.6 [22.9 kB] Get: 29 http://deb.debian.org/debian trixie/main arm64 po-debconf all 1.0.21+nmu1 [248 kB] Get: 30 http://deb.debian.org/debian trixie/main arm64 debhelper all 13.15.3 [901 kB] -Fetched 18.6 MB in 0s (126 MB/s) +Fetched 18.6 MB in 0s (135 MB/s) debconf: delaying package configuration, since apt-utils is not installed Selecting previously unselected package sensible-utils. (Reading database ... (Reading database ... 5% (Reading database ... 10% (Reading database ... 15% (Reading database ... 20% (Reading database ... 25% (Reading database ... 30% (Reading database ... 35% (Reading database ... 40% (Reading database ... 45% (Reading database ... 50% (Reading database ... 55% (Reading database ... 60% (Reading database ... 65% (Reading database ... 70% (Reading database ... 75% (Reading database ... 80% (Reading database ... 85% (Reading database ... 90% (Reading database ... 95% (Reading database ... 100% (Reading database ... 19937 files and directories currently installed.) @@ -277,7 +309,11 @@ Building tag database... -> Finished parsing the build-deps I: Building the package -I: Running cd /build/reproducible-path/wham-align-0.1.5/ && env PATH="/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" HOME="/nonexistent/first-build" dpkg-buildpackage -us -uc -b && env PATH="/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" HOME="/nonexistent/first-build" dpkg-genchanges -S > ../wham-align_0.1.5-8_source.changes +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/A99_set_merged_usr starting +Not re-configuring usrmerge for trixie +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/A99_set_merged_usr finished +hostname: Name or service not known +I: Running cd /build/reproducible-path/wham-align-0.1.5/ && env PATH="/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/i/capture/the/path" HOME="/nonexistent/second-build" dpkg-buildpackage -us -uc -b && env PATH="/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/i/capture/the/path" HOME="/nonexistent/second-build" dpkg-genchanges -S > ../wham-align_0.1.5-8_source.changes dpkg-buildpackage: info: source package wham-align dpkg-buildpackage: info: source version 0.1.5-8 dpkg-buildpackage: info: source distribution unstable @@ -333,18 +369,43 @@ | ^~~~~~ g++ -c -g -O2 -ffile-prefix-map=/build/reproducible-path/wham-align-0.1.5=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wall -pedantic -fsigned-char -O3 -Wno-write-strings writer.cpp g++ -c -g -O2 -ffile-prefix-map=/build/reproducible-path/wham-align-0.1.5=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wall -pedantic -fsigned-char -O3 -Wno-write-strings builder.cpp -short.cpp: In member function 'int ShortRead::preProcess(int)': -short.cpp:212:16: warning: suggest parentheses around assignment used as truth value [-Wparentheses] - 212 | while (len = readLine(file, str, 256)) { - | ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~ -short.cpp: In member function 'ShortRead** ShortRead::split(int)': -short.cpp:483:21: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare] - 483 | if (szPartition <= nRead - curRead) - | ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ +model.cpp: In static member function 'static int AlignerModel::estimateNumPartition(unsigned int, int, int, bool)': +model.cpp:76:16: warning: variable 'nIndex' set but not used [-Wunused-but-set-variable] + 76 | unsigned int nIndex, nLookup, nBucket; + | ^~~~~~ interval.cpp: In constructor 'IntervalTree::IntervalTree(uint32, uint32)': interval.cpp:67:21: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] 67 | for (int i = 0; i < numLevel - 1; i++) { | ~~^~~~~~~~~~~~~~ +model.cpp: In static member function 'static unsigned int AlignerModel::getFreeMemory()': +model.cpp:166:28: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] + 166 | fscanf(file, "MemTotal: %d kB\n", &memTotal); + | ~^ ~~~~~~~~~ + | | | + | int* unsigned int* + | %d +model.cpp:167:27: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] + 167 | fscanf(file, "MemFree: %d kB\n", &memFree); + | ~^ ~~~~~~~~ + | | | + | int* unsigned int* + | %d +model.cpp:168:27: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] + 168 | fscanf(file, "Buffers: %d kB\n", &memBuffer); + | ~^ ~~~~~~~~~~ + | | | + | int* unsigned int* + | %d +model.cpp:169:26: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] + 169 | fscanf(file, "Cached: %d kB\n", &memCache); + | ~^ ~~~~~~~~~ + | | | + | int* unsigned int* + | %d +model.cpp: In static member function 'static unsigned int AlignerModel::estimateHashtableSpace(unsigned int, int, int, int)': +model.cpp:198:26: warning: variable 'p2' set but not used [-Wunused-but-set-variable] + 198 | double nSpace, p0, p1, p2; + | ^~ interval.cpp: In member function 'int IntervalTree::save(char*)': interval.cpp:304:11: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] 304 | if (ret != numLEntry) { @@ -359,20 +420,31 @@ interval.cpp:370:11: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] 370 | if (ret != numIEntry) { | ~~~~^~~~~~~~~~~~ +short.cpp: In member function 'int ShortRead::preProcess(int)': +short.cpp:212:16: warning: suggest parentheses around assignment used as truth value [-Wparentheses] + 212 | while (len = readLine(file, str, 256)) { + | ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~ +short.cpp: In member function 'ShortRead** ShortRead::split(int)': +short.cpp:483:21: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare] + 483 | if (szPartition <= nRead - curRead) + | ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ In file included from hitset.h:31, from hitset.cpp:1: pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] 265 | if (num > nError) { | ~~~~^~~~~~~~ +In file included from hitset.h:31, + from writer.h:26, + from writer.cpp:21: +pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': +pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] + 265 | if (num > nError) { + | ~~~~^~~~~~~~ pair.h: In static member function 'static bool PairAligner::pairAlignDP(int64*, int64*, int, int, int, bool)': pair.h:602:15: warning: unused variable 'offset' [-Wunused-variable] 602 | int i, j, offset, min, tmp; | ^~~~~~ -model.cpp: In static member function 'static int AlignerModel::estimateNumPartition(unsigned int, int, int, bool)': -model.cpp:76:16: warning: variable 'nIndex' set but not used [-Wunused-but-set-variable] - 76 | unsigned int nIndex, nLookup, nBucket; - | ^~~~~~ pair.h: In static member function 'static bool PairAligner::pairAlignBitVector64(int64*, int64*, int, int, int)': pair.h:682:33: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] 682 | register unsigned long long P, M, X, U, Y; @@ -389,31 +461,6 @@ pair.h:682:45: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] 682 | register unsigned long long P, M, X, U, Y; | ^ -model.cpp: In static member function 'static unsigned int AlignerModel::getFreeMemory()': -model.cpp:166:28: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] - 166 | fscanf(file, "MemTotal: %d kB\n", &memTotal); - | ~^ ~~~~~~~~~ - | | | - | int* unsigned int* - | %d -model.cpp:167:27: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] - 167 | fscanf(file, "MemFree: %d kB\n", &memFree); - | ~^ ~~~~~~~~ - | | | - | int* unsigned int* - | %d -model.cpp:168:27: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] - 168 | fscanf(file, "Buffers: %d kB\n", &memBuffer); - | ~^ ~~~~~~~~~~ - | | | - | int* unsigned int* - | %d -model.cpp:169:26: warning: format '%d' expects argument of type 'int*', but argument 3 has type 'unsigned int*' [-Wformat=] - 169 | fscanf(file, "Cached: %d kB\n", &memCache); - | ~^ ~~~~~~~~~ - | | | - | int* unsigned int* - | %d pair.h:684:15: warning: unused variable 'num' [-Wunused-variable] 684 | int i, p, num, base, Cscore; | ^~~ @@ -423,10 +470,6 @@ pair.h:685:9: warning: unused variable 'a' [-Wunused-variable] 685 | int a; | ^ -model.cpp: In static member function 'static unsigned int AlignerModel::estimateHashtableSpace(unsigned int, int, int, int)': -model.cpp:198:26: warning: variable 'p2' set but not used [-Wunused-but-set-variable] - 198 | double nSpace, p0, p1, p2; - | ^~ pair.h: In static member function 'static bool PairAligner::pairAlignBitVector(int64*, int64*, int, int, int)': pair.h:755:15: warning: unused variable 'num' [-Wunused-variable] 755 | int i, p, num, base, Cscore; @@ -441,21 +484,6 @@ pair.h:974:23: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses] 974 | tmp.high = x.high ^ - x.high | (0LL - (x.low != 0)); | ~~~~~~~^~~~~~~~~~ -hitset.h: In constructor 'HitSet::HitSet(int, int, int, bool, bool)': -hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] - 86 | int size; - | ^~~~ -hitset.cpp: In member function 'int HitPairSet::build(HitSet*, HitSet*, uint32, uint32, bool (*)[2], bool)': -hitset.cpp:90:8: warning: unused variable 'pairMatch' [-Wunused-variable] - 90 | bool pairMatch = false; - | ^~~~~~~~~ -In file included from hitset.h:31, - from writer.h:26, - from writer.cpp:21: -pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': -pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] - 265 | if (num > nError) { - | ~~~~^~~~~~~~ pair.h: In static member function 'static bool PairAligner::pairAlignDP(int64*, int64*, int, int, int, bool)': pair.h:602:15: warning: unused variable 'offset' [-Wunused-variable] 602 | int i, j, offset, min, tmp; @@ -485,6 +513,10 @@ pair.h:685:9: warning: unused variable 'a' [-Wunused-variable] 685 | int a; | ^ +hitset.h: In constructor 'HitSet::HitSet(int, int, int, bool, bool)': +hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] + 86 | int size; + | ^~~~ pair.h: In static member function 'static bool PairAligner::pairAlignBitVector(int64*, int64*, int, int, int)': pair.h:755:15: warning: unused variable 'num' [-Wunused-variable] 755 | int i, p, num, base, Cscore; @@ -503,6 +535,10 @@ hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] 86 | int size; | ^~~~ +hitset.cpp: In member function 'int HitPairSet::build(HitSet*, HitSet*, uint32, uint32, bool (*)[2], bool)': +hitset.cpp:90:8: warning: unused variable 'pairMatch' [-Wunused-variable] + 90 | bool pairMatch = false; + | ^~~~~~~~~ writer.cpp: In member function 'void SimpleWriter::writeField(int64*, int64*, strand)': writer.cpp:154:27: warning: suggest parentheses around '-' in operand of '&' [-Wparentheses] 154 | j = BITS_PER_LONGWORD - (length * BITS_PER_BASE) & BITS_LONGWORD_MASK; @@ -512,6 +548,13 @@ writer.cpp:148:19: warning: unused variable 'soffset' [-Wunused-variable] 148 | uint32 sid = 0, soffset = 0; | ^~~~~~~ +In file included from hitset.h:31, + from hash.h:29, + from hash.cpp:27: +pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': +pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] + 265 | if (num > nError) { + | ~~~~^~~~~~~~ writer.cpp: In member function 'void SamWriter::writeValidAlignment(int, HitSet*)': writer.cpp:271:7: warning: unused variable 'i' [-Wunused-variable] 271 | int i, j, k, h, lastDiff; @@ -555,13 +598,6 @@ writer.cpp:619:47: warning: suggest parentheses around comparison in operand of '&' [-Wparentheses] 619 | if (!mate1Unmatch && !mate2Unmatch & sid1 == sid2) { | ~~~~~^~~~~~~ -In file included from hitset.h:31, - from hash.h:29, - from embedhash.cpp:27: -pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': -pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] - 265 | if (num > nError) { - | ~~~~^~~~~~~~ pair.h: In static member function 'static bool PairAligner::pairAlignDP(int64*, int64*, int, int, int, bool)': pair.h:602:15: warning: unused variable 'offset' [-Wunused-variable] 602 | int i, j, offset, min, tmp; @@ -615,15 +651,7 @@ | ^~~ In file included from hitset.h:31, from hash.h:29, - from aligner.h:25, - from aligner.cpp:25: -pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': -pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] - 265 | if (num > nError) { - | ~~~~^~~~~~~~ -In file included from hitset.h:31, - from hash.h:29, - from hash.cpp:27: + from embedhash.cpp:27: pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] 265 | if (num > nError) { @@ -675,57 +703,6 @@ hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] 86 | int size; | ^~~~ -pair.h: In static member function 'static bool PairAligner::pairAlignDP(int64*, int64*, int, int, int, bool)': -pair.h:602:15: warning: unused variable 'offset' [-Wunused-variable] - 602 | int i, j, offset, min, tmp; - | ^~~~~~ -pair.h: In static member function 'static bool PairAligner::pairAlignBitVector64(int64*, int64*, int, int, int)': -pair.h:682:33: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] - 682 | register unsigned long long P, M, X, U, Y; - | ^ -pair.h:682:36: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] - 682 | register unsigned long long P, M, X, U, Y; - | ^ -pair.h:682:39: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] - 682 | register unsigned long long P, M, X, U, Y; - | ^ -pair.h:682:42: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] - 682 | register unsigned long long P, M, X, U, Y; - | ^ -pair.h:682:45: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] - 682 | register unsigned long long P, M, X, U, Y; - | ^ -pair.h:684:15: warning: unused variable 'num' [-Wunused-variable] - 684 | int i, p, num, base, Cscore; - | ^~~ -pair.h:684:20: warning: unused variable 'base' [-Wunused-variable] - 684 | int i, p, num, base, Cscore; - | ^~~~ -pair.h:685:9: warning: unused variable 'a' [-Wunused-variable] - 685 | int a; - | ^ -pair.h: In static member function 'static bool PairAligner::pairAlignBitVector(int64*, int64*, int, int, int)': -pair.h:755:15: warning: unused variable 'num' [-Wunused-variable] - 755 | int i, p, num, base, Cscore; - | ^~~ -pair.h:755:20: warning: unused variable 'base' [-Wunused-variable] - 755 | int i, p, num, base, Cscore; - | ^~~~ -pair.h:756:9: warning: unused variable 'a' [-Wunused-variable] - 756 | int a; - | ^ -hash.h: In function 'uint32 murmurHash(int64*, uint32)': -hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] - 86 | uint32 sum = 0; - | ^~~ -pair.h: In static member function 'static int128 PairAligner::Bit128RemoveSmearRightmost1(int128)': -pair.h:974:23: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses] - 974 | tmp.high = x.high ^ - x.high | (0LL - (x.low != 0)); - | ~~~~~~~^~~~~~~~~~ -hitset.h: In constructor 'HitSet::HitSet(int, int, int, bool, bool)': -hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] - 86 | int size; - | ^~~~ hash.h: In function 'uint32 murmurHash(int64*, uint32)': hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] 86 | uint32 sum = 0; @@ -776,7 +753,7 @@ In file included from hitset.h:31, from hash.h:29, from aligner.h:25, - from builder.cpp:31: + from aligner.cpp:25: pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] 265 | if (num > nError) { @@ -835,7 +812,7 @@ In file included from hitset.h:31, from hash.h:29, from aligner.h:25, - from main.cpp:34: + from builder.cpp:31: pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] 265 | if (num > nError) { @@ -887,18 +864,89 @@ hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] 86 | int size; | ^~~~ +hash.h: In function 'uint32 murmurHash(int64*, uint32)': +hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] + 86 | uint32 sum = 0; + | ^~~ +hash.cpp: In member function 'int HashTable::preProcessEnd()': +hash.cpp:236:11: warning: comparison of integer expressions of different signedness: 'int' and 'const uint32' {aka 'const unsigned int'} [-Wsign-compare] + 236 | if (h >= nHistogram) + | ~~^~~~~~~~~~~~~ +hash.cpp:285:33: warning: comparison of integer expressions of different signedness: 'uint32' {aka 'unsigned int'} and 'int' [-Wsign-compare] + 285 | if (useEmbedTables() && tmp > embedShreshold) { + | ~~~~^~~~~~~~~~~~~~~~ +hash.cpp: In member function 'void HashTable::setScanThreshold(double)': +hash.cpp:343:21: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare] + 343 | for (int i = 0; i < nHistogram - 1; i++) { + | ~~^~~~~~~~~~~~~~~~ +hash.cpp: In member function 'void HashTable::insert(int64*, unsigned int)': +hash.cpp:460:38: warning: operation on '*(((HashTable*)this)->HashTable::overflowPool + ((sizetype)(((long unsigned int)curOverflowEntry) * 4)))' may be undefined [-Wsequence-point] + 460 | overflowPool[curOverflowEntry] = +hash.cpp:411:10: warning: unused variable 'seqOffset' [-Wunused-variable] + 411 | uint32 seqOffset; + | ^~~~~~~~~ +hash.cpp:413:10: warning: unused variable 'counter' [-Wunused-variable] + 413 | uint32 counter = 0; + | ^~~~~~~ +hash.cpp:414:11: warning: variable 'seqVector' set but not used [-Wunused-but-set-variable] + 414 | int64 * seqVector; + | ^~~~~~~~~ +hash.cpp:416:11: warning: unused variable 'target' [-Wunused-variable] + 416 | int64 * target = &tspace[8]; + | ^~~~~~ +hash.cpp:417:8: warning: unused variable 'isBloomFilter' [-Wunused-variable] + 417 | bool isBloomFilter; + | ^~~~~~~~~~~~~ +hash.cpp:418:12: warning: unused variable 'bloomFilter' [-Wunused-variable] + 418 | uint32 * bloomFilter; + | ^~~~~~~~~~~ +hash.cpp:419:10: warning: unused variable 'bloomFilterNum' [-Wunused-variable] + 419 | uint32 bloomFilterNum; + | ^~~~~~~~~~~~~~ In file included from hitset.h:31, from hash.h:29, from aligner.h:25, - from sequence.cpp:30: + from main.cpp:34: pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] 265 | if (num > nError) { | ~~~~^~~~~~~~ -hash.h: In function 'uint32 murmurHash(int64*, uint32)': -hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] - 86 | uint32 sum = 0; - | ^~~ +hash.cpp: In member function 'int HashTable::buildEmbedTable()': +hash.cpp:469:33: warning: unused variable 'rest' [-Wunused-variable] + 469 | int64 * query = &ttspace[8], *rest = &ttspace[24], *rest1 = &ttspace[40], + | ^~~~ +hash.cpp:469:55: warning: unused variable 'rest1' [-Wunused-variable] + 469 | int64 * query = &ttspace[8], *rest = &ttspace[24], *rest1 = &ttspace[40], + | ^~~~~ +hash.cpp:470:8: warning: unused variable 'rest2' [-Wunused-variable] + 470 | *rest2 = &ttspace[56], *noheadkey = &ttspace[72]; + | ^~~~~ +hash.cpp:470:31: warning: unused variable 'noheadkey' [-Wunused-variable] + 470 | *rest2 = &ttspace[56], *noheadkey = &ttspace[72]; + | ^~~~~~~~~ +hash.cpp:472:34: warning: variable 'startOffset' set but not used [-Wunused-but-set-variable] + 472 | uint32 seqOffset, entryOffset, startOffset; + | ^~~~~~~~~~~ +hash.cpp: In member function 'void HashTable::preinsertEmbedTableEntry(int64*, int, int)': +hash.cpp:587:11: warning: unused variable 'noheadkey' [-Wunused-variable] + 587 | int64 * noheadkey = &tspace[8], *rest = &tspace[24]; + | ^~~~~~~~~ +hash.cpp:590:8: warning: unused variable 'subkey4' [-Wunused-variable] + 590 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; + | ^~~~~~~ +hash.cpp:590:32: warning: unused variable 'subkey5' [-Wunused-variable] + 590 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; + | ^~~~~~~ +hash.cpp: In member function 'void HashTable::insertEmbedTableEntry(int64*, uint32, int, int)': +hash.cpp:662:11: warning: unused variable 'noheadkey' [-Wunused-variable] + 662 | int64 * noheadkey = &tspace[8], *rest = &tspace[24]; + | ^~~~~~~~~ +hash.cpp:665:8: warning: unused variable 'subkey4' [-Wunused-variable] + 665 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; + | ^~~~~~~ +hash.cpp:665:32: warning: unused variable 'subkey5' [-Wunused-variable] + 665 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; + | ^~~~~~~ pair.h: In static member function 'static bool PairAligner::pairAlignDP(int64*, int64*, int, int, int, bool)': pair.h:602:15: warning: unused variable 'offset' [-Wunused-variable] 602 | int i, j, offset, min, tmp; @@ -916,9 +964,22 @@ pair.h:682:42: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] 682 | register unsigned long long P, M, X, U, Y; | ^ +hash.cpp: In member function 'unsigned int HashTable::lookup(int64*, int64*, int, char*, strand, int, HitSet*, bool)': +hash.cpp:746:7: warning: unused variable 'num' [-Wunused-variable] + 746 | int num; + | ^~~ +hash.cpp:750:11: warning: unused variable 'diff' [-Wunused-variable] + 750 | int64 * diff = &tspace1[8], *target = &tspace2[8]; + | ^~~~ pair.h:682:45: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] 682 | register unsigned long long P, M, X, U, Y; | ^ +hash.cpp:753:10: warning: unused variable 'sid' [-Wunused-variable] + 753 | uint32 sid, soffset; + | ^~~ +hash.cpp:753:15: warning: unused variable 'soffset' [-Wunused-variable] + 753 | uint32 sid, soffset; + | ^~~~~~~ pair.h:684:15: warning: unused variable 'num' [-Wunused-variable] 684 | int i, p, num, base, Cscore; | ^~~ @@ -928,6 +989,19 @@ pair.h:685:9: warning: unused variable 'a' [-Wunused-variable] 685 | int a; | ^ +hash.cpp: In member function 'unsigned int HashTable::lookupEmbedTable(int64*, uint32, int, char*, strand, int, HitSet*, bool)': +hash.cpp:962:11: warning: unused variable 'noheadkey' [-Wunused-variable] + 962 | int64 * noheadkey = &tspace[8], *rest = &tspace[24]; + | ^~~~~~~~~ +hash.cpp:965:8: warning: unused variable 'subkey4' [-Wunused-variable] + 965 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; + | ^~~~~~~ +hash.cpp:965:32: warning: unused variable 'subkey5' [-Wunused-variable] + 965 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; + | ^~~~~~~ +hash.cpp:973:11: warning: unused variable 'seqVector' [-Wunused-variable] + 973 | int64 * seqVector = sequence->getSequence(); + | ^~~~~~~~~ pair.h: In static member function 'static bool PairAligner::pairAlignBitVector(int64*, int64*, int, int, int)': pair.h:755:15: warning: unused variable 'num' [-Wunused-variable] 755 | int i, p, num, base, Cscore; @@ -938,14 +1012,65 @@ pair.h:756:9: warning: unused variable 'a' [-Wunused-variable] 756 | int a; | ^ +hash.cpp: In member function 'bool HashTable::lookup(int64*, uint32)': +hash.cpp:1052:7: warning: unused variable 'num' [-Wunused-variable] + 1052 | int num; + | ^~~ +hash.cpp:1056:11: warning: unused variable 'diff' [-Wunused-variable] + 1056 | int64 * diff = &tspace1[8], *target = &tspace2[8]; + | ^~~~ +hash.cpp:1056:32: warning: unused variable 'target' [-Wunused-variable] + 1056 | int64 * diff = &tspace1[8], *target = &tspace2[8]; + | ^~~~~~ +hash.cpp:1057:11: warning: variable 'seqVector' set but not used [-Wunused-but-set-variable] + 1057 | int64 * seqVector; + | ^~~~~~~~~ +hash.cpp:1059:10: warning: unused variable 'sid' [-Wunused-variable] + 1059 | uint32 sid, soffset; + | ^~~ +hash.cpp:1059:15: warning: unused variable 'soffset' [-Wunused-variable] + 1059 | uint32 sid, soffset; + | ^~~~~~~ pair.h: In static member function 'static int128 PairAligner::Bit128RemoveSmearRightmost1(int128)': pair.h:974:23: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses] 974 | tmp.high = x.high ^ - x.high | (0LL - (x.low != 0)); | ~~~~~~~^~~~~~~~~~ +hash.cpp: In member function 'uint32 HashTable::printOverflowList(uint32, uint32, int64*)': +hash.cpp:1184:23: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] + 1184 | for (int i = 0; i < keyOffset / BITS_PER_BASE; i++) +hash.cpp:1188:23: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] + 1188 | for (int i = 0; i < (lenSeq - keyOffset - lenKey) / BITS_PER_BASE; i++) +hash.cpp:1247:27: warning: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'long unsigned int'} [-Wsign-compare] + 1247 | for (int j = 0; j < strlen(set[i].str); j++) + | ~~^~~~~~~~~~~~~~~~~~~~ hitset.h: In constructor 'HitSet::HitSet(int, int, int, bool, bool)': hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] 86 | int size; | ^~~~ +hash.cpp:1254:25: warning: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'long unsigned int'} [-Wsign-compare] + 1254 | for (int j = 0; j < strlen(set[i].str); j++) + | ~~^~~~~~~~~~~~~~~~~~~~ +hash.cpp:1137:11: warning: unused variable 'diff' [-Wunused-variable] + 1137 | int64 * diff = &tspace1[8], *target = &tspace2[8]; + | ^~~~ +hash.cpp: In member function 'int HashTable::sortList()': +hash.cpp:1430:14: warning: variable 'pool' set but not used [-Wunused-but-set-variable] + 1430 | uint32 * pool; + | ^~~~ +hash.cpp:1321:34: warning: variable 'offset' set but not used [-Wunused-but-set-variable] + 1321 | uint32 entryOffset, seqOffset, offset; + | ^~~~~~ +hash.h: In function 'uint32 murmurHash(int64*, uint32)': +hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] + 86 | uint32 sum = 0; + | ^~~ +hash.cpp: In member function 'void HashTable::checkRepeat(uint32)': +hash.cpp:1673:34: warning: variable 'offset' set but not used [-Wunused-but-set-variable] + 1673 | uint32 entryOffset, seqOffset, offset; + | ^~~~~~ +hash.cpp:1690:5: warning: label 'restart' defined but not used [-Wunused-label] + 1690 | restart: if (buckets[bucketId] == HASH_EMPTY + | ^~~~~~~ embedhash.cpp: In member function 'int EmbedHashTable::preProcessEnd()': embedhash.cpp:137:16: warning: unused variable 'ln2' [-Wunused-variable] 137 | const double ln2 = log(2); @@ -963,10 +1088,6 @@ embedhash.cpp:328:10: warning: unused variable 'seqOffset' [-Wunused-variable] 328 | uint32 seqOffset; | ^~~~~~~~~ -hash.h: In function 'uint32 murmurHash(int64*, uint32)': -hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] - 86 | uint32 sum = 0; - | ^~~ embedhash.cpp:330:10: warning: unused variable 'counter' [-Wunused-variable] 330 | uint32 counter = 0; | ^~~~~~~ @@ -1001,6 +1122,65 @@ embedhash.cpp:392:15: warning: unused variable 'soffset' [-Wunused-variable] 392 | uint32 sid, soffset; | ^~~~~~~ +In file included from hitset.h:31, + from hash.h:29, + from aligner.h:25, + from sequence.cpp:30: +pair.h: In static member function 'static ErrorVector PairAligner::pairAlignDBA(int64*, int64*, int, int, int)': +pair.h:265:13: warning: comparison of integer expressions of different signedness: 'int64' {aka 'long long unsigned int'} and 'int' [-Wsign-compare] + 265 | if (num > nError) { + | ~~~~^~~~~~~~ +pair.h: In static member function 'static bool PairAligner::pairAlignDP(int64*, int64*, int, int, int, bool)': +pair.h:602:15: warning: unused variable 'offset' [-Wunused-variable] + 602 | int i, j, offset, min, tmp; + | ^~~~~~ +pair.h: In static member function 'static bool PairAligner::pairAlignBitVector64(int64*, int64*, int, int, int)': +pair.h:682:33: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] + 682 | register unsigned long long P, M, X, U, Y; + | ^ +pair.h:682:36: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] + 682 | register unsigned long long P, M, X, U, Y; + | ^ +pair.h:682:39: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] + 682 | register unsigned long long P, M, X, U, Y; + | ^ +pair.h:682:42: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] + 682 | register unsigned long long P, M, X, U, Y; + | ^ +pair.h:682:45: warning: ISO C++17 does not allow 'register' storage class specifier [-Wregister] + 682 | register unsigned long long P, M, X, U, Y; + | ^ +pair.h:684:15: warning: unused variable 'num' [-Wunused-variable] + 684 | int i, p, num, base, Cscore; + | ^~~ +pair.h:684:20: warning: unused variable 'base' [-Wunused-variable] + 684 | int i, p, num, base, Cscore; + | ^~~~ +pair.h:685:9: warning: unused variable 'a' [-Wunused-variable] + 685 | int a; + | ^ +pair.h: In static member function 'static bool PairAligner::pairAlignBitVector(int64*, int64*, int, int, int)': +pair.h:755:15: warning: unused variable 'num' [-Wunused-variable] + 755 | int i, p, num, base, Cscore; + | ^~~ +pair.h:755:20: warning: unused variable 'base' [-Wunused-variable] + 755 | int i, p, num, base, Cscore; + | ^~~~ +pair.h:756:9: warning: unused variable 'a' [-Wunused-variable] + 756 | int a; + | ^ +pair.h: In static member function 'static int128 PairAligner::Bit128RemoveSmearRightmost1(int128)': +pair.h:974:23: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses] + 974 | tmp.high = x.high ^ - x.high | (0LL - (x.low != 0)); + | ~~~~~~~^~~~~~~~~~ +hitset.h: In constructor 'HitSet::HitSet(int, int, int, bool, bool)': +hitset.h:86:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] + 86 | int size; + | ^~~~ +hash.h: In function 'uint32 murmurHash(int64*, uint32)': +hash.h:86:10: warning: unused variable 'sum' [-Wunused-variable] + 86 | uint32 sum = 0; + | ^~~ sequence.cpp: In member function 'void CompactSequence::extractFileName(char*, char*)': sequence.cpp:97:10: warning: unused variable 'str' [-Wunused-variable] 97 | char * str; @@ -1066,6 +1246,63 @@ sequence.cpp:804:11: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] 804 | if (ret != size) { | ~~~~^~~~~~~ +main.cpp: In function 'int getArguVal(char*)': +main.cpp:159:15: warning: suggest parentheses around assignment used as truth value [-Wparentheses] + 159 | else if (*c = 'g' || *c == 'G') + | ~~~^~~~~~~~~~~~~~~~~~ +main.cpp: In function 'int align(char*, AlignInfo*, char*, int)': +main.cpp:350:7: warning: unused variable 'i' [-Wunused-variable] + 350 | int i, j, ret; + | ^ +main.cpp:350:10: warning: unused variable 'j' [-Wunused-variable] + 350 | int i, j, ret; + | ^ +main.cpp: In function 'int main(int, char**)': +main.cpp:523:7: warning: unused variable 'numKeys' [-Wunused-variable] + 523 | int numKeys; + | ^~~~~~~ +main.cpp:526:7: warning: unused variable 'nSeq' [-Wunused-variable] + 526 | int nSeq = 1; + | ^~~~ +main.cpp:532:21: warning: unused variable 'sequence' [-Wunused-variable] + 532 | CompactSequence * sequence; + | ^~~~~~~~ +main.cpp:534:15: warning: unused variable 'threadpool' [-Wunused-variable] + 534 | pthread_t * threadpool; + | ^~~~~~~~~~ +main.cpp:536:9: warning: unused variable 'threadid' [-Wunused-variable] + 536 | int * threadid; + | ^~~~~~~~ +hash.cpp: In member function 'unsigned int HashTable::lookupEmbedTable(int64*, uint32, int, char*, strand, int, HitSet*, bool)': +hash.cpp:1049:1: warning: control reaches end of non-void function [-Wreturn-type] + 1049 | } + | ^ +In file included from hash.cpp:31: +edit_distance.h: At global scope: +edit_distance.h:25:18: warning: 'obj_score_mat' defined but not used [-Wunused-variable] + 25 | static score_mat obj_score_mat = { 1, 1, 1, 1 }; + | ^~~~~~~~~~~~~ +In file included from pair.h:26: +rdtsc.h: In function 'long long unsigned int curtick()': +rdtsc.h:32:10: warning: 'tick' is used uninitialized [-Wuninitialized] + 32 | return tick; + | ^~~~ +rdtsc.h:20:22: note: 'tick' was declared here + 20 | unsigned long long tick; + | ^~~~ +In file included from embedhash.cpp:32: +edit_distance.h: At global scope: +edit_distance.h:25:18: warning: 'obj_score_mat' defined but not used [-Wunused-variable] + 25 | static score_mat obj_score_mat = { 1, 1, 1, 1 }; + | ^~~~~~~~~~~~~ +In file included from pair.h:26: +rdtsc.h: In function 'long long unsigned int curtick()': +rdtsc.h:32:10: warning: 'tick' is used uninitialized [-Wuninitialized] + 32 | return tick; + | ^~~~ +rdtsc.h:20:22: note: 'tick' was declared here + 20 | unsigned long long tick; + | ^~~~ aligner.cpp: In member function 'int Aligner::initLookupArray()': aligner.cpp:365:37: warning: suggest parentheses around comparison in operand of '&' [-Wparentheses] 365 | if ((indexBits[i] >> j) & 0x1 == 1) { @@ -1077,16 +1314,9 @@ aligner.cpp:534:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 534 | for (i = 0; i < nHashTable; i++) { | ~~^~~~~~~~~~~~ -hash.cpp: In member function 'int HashTable::preProcessEnd()': -hash.cpp:236:11: warning: comparison of integer expressions of different signedness: 'int' and 'const uint32' {aka 'const unsigned int'} [-Wsign-compare] - 236 | if (h >= nHistogram) - | ~~^~~~~~~~~~~~~ aligner.cpp:712:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 712 | for (i = 0; i < nHashTable; i++) { | ~~^~~~~~~~~~~~ -hash.cpp:285:33: warning: comparison of integer expressions of different signedness: 'uint32' {aka 'unsigned int'} and 'int' [-Wsign-compare] - 285 | if (useEmbedTables() && tmp > embedShreshold) { - | ~~~~^~~~~~~~~~~~~~~~ aligner.cpp: In member function 'int Aligner::build(char*)': aligner.cpp:724:16: warning: unused variable 'i' [-Wunused-variable] 724 | unsigned int i, j; @@ -1098,10 +1328,6 @@ aligner.cpp:807:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 807 | for (i = 0; i < nHashTable; i++) { | ~~^~~~~~~~~~~~ -hash.cpp: In member function 'void HashTable::setScanThreshold(double)': -hash.cpp:343:21: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare] - 343 | for (int i = 0; i < nHistogram - 1; i++) { - | ~~^~~~~~~~~~~~~~~~ aligner.cpp:844:23: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 844 | for (i1 = 0; i1 < lenPartition * (nPartition - 1); i1 += lenPartition) { | ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ @@ -1126,36 +1352,12 @@ aligner.cpp:930:26: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 930 | for (i2 = i1; i2 < lenPartition * (nPartition - 4); | ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -hash.cpp: In member function 'void HashTable::insert(int64*, unsigned int)': -hash.cpp:460:38: warning: operation on '*(((HashTable*)this)->HashTable::overflowPool + ((sizetype)(((long unsigned int)curOverflowEntry) * 4)))' may be undefined [-Wsequence-point] - 460 | overflowPool[curOverflowEntry] = aligner.cpp:933:28: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 933 | for (i3 = i2; i3 < lenPartition * (nPartition - 4); | ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -hash.cpp:411:10: warning: unused variable 'seqOffset' [-Wunused-variable] - 411 | uint32 seqOffset; - | ^~~~~~~~~ -hash.cpp:413:10: warning: unused variable 'counter' [-Wunused-variable] - 413 | uint32 counter = 0; - | ^~~~~~~ -hash.cpp:414:11: warning: variable 'seqVector' set but not used [-Wunused-but-set-variable] - 414 | int64 * seqVector; - | ^~~~~~~~~ aligner.cpp:936:30: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 936 | for (i4 = i3; i4 < lenPartition * (nPartition - 4); i4 += | ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -hash.cpp:416:11: warning: unused variable 'target' [-Wunused-variable] - 416 | int64 * target = &tspace[8]; - | ^~~~~~ -hash.cpp:417:8: warning: unused variable 'isBloomFilter' [-Wunused-variable] - 417 | bool isBloomFilter; - | ^~~~~~~~~~~~~ -hash.cpp:418:12: warning: unused variable 'bloomFilter' [-Wunused-variable] - 418 | uint32 * bloomFilter; - | ^~~~~~~~~~~ -hash.cpp:419:10: warning: unused variable 'bloomFilterNum' [-Wunused-variable] - 419 | uint32 bloomFilterNum; - | ^~~~~~~~~~~~~~ aligner.cpp:951:23: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 951 | for (i1 = 0; i1 < lenPartition * (nPartition - 5); i1 += lenPartition) { | ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ @@ -1177,22 +1379,6 @@ aligner.cpp:1006:19: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 1006 | for (i = 0; i < nHashTable; i++) { | ~~^~~~~~~~~~~~ -hash.cpp: In member function 'int HashTable::buildEmbedTable()': -hash.cpp:469:33: warning: unused variable 'rest' [-Wunused-variable] - 469 | int64 * query = &ttspace[8], *rest = &ttspace[24], *rest1 = &ttspace[40], - | ^~~~ -hash.cpp:469:55: warning: unused variable 'rest1' [-Wunused-variable] - 469 | int64 * query = &ttspace[8], *rest = &ttspace[24], *rest1 = &ttspace[40], - | ^~~~~ -hash.cpp:470:8: warning: unused variable 'rest2' [-Wunused-variable] - 470 | *rest2 = &ttspace[56], *noheadkey = &ttspace[72]; - | ^~~~~ -hash.cpp:470:31: warning: unused variable 'noheadkey' [-Wunused-variable] - 470 | *rest2 = &ttspace[56], *noheadkey = &ttspace[72]; - | ^~~~~~~~~ -hash.cpp:472:34: warning: variable 'startOffset' set but not used [-Wunused-but-set-variable] - 472 | uint32 seqOffset, entryOffset, startOffset; - | ^~~~~~~~~~~ aligner.cpp: In member function 'unsigned int Aligner::alignFirst(int64*, char*, strand, int, HitSet*, bool)': aligner.cpp:1035:10: warning: unused variable 'offset' [-Wunused-variable] 1035 | uint32 offset; @@ -1200,16 +1386,6 @@ aligner.cpp:1039:7: warning: unused variable 'nHit' [-Wunused-variable] 1039 | int nHit = 0; | ^~~~ -hash.cpp: In member function 'void HashTable::preinsertEmbedTableEntry(int64*, int, int)': -hash.cpp:587:11: warning: unused variable 'noheadkey' [-Wunused-variable] - 587 | int64 * noheadkey = &tspace[8], *rest = &tspace[24]; - | ^~~~~~~~~ -hash.cpp:590:8: warning: unused variable 'subkey4' [-Wunused-variable] - 590 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; - | ^~~~~~~ -hash.cpp:590:32: warning: unused variable 'subkey5' [-Wunused-variable] - 590 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; - | ^~~~~~~ aligner.cpp: In member function 'unsigned int Aligner::align(int64*, char*, strand, int, HitSet*, bool, bool)': aligner.cpp:1119:10: warning: variable 'offset' set but not used [-Wunused-but-set-variable] 1119 | uint32 offset; @@ -1217,16 +1393,6 @@ aligner.cpp:1123:7: warning: variable 'nHit' set but not used [-Wunused-but-set-variable] 1123 | int nHit = 0; | ^~~~ -hash.cpp: In member function 'void HashTable::insertEmbedTableEntry(int64*, uint32, int, int)': -hash.cpp:662:11: warning: unused variable 'noheadkey' [-Wunused-variable] - 662 | int64 * noheadkey = &tspace[8], *rest = &tspace[24]; - | ^~~~~~~~~ -hash.cpp:665:8: warning: unused variable 'subkey4' [-Wunused-variable] - 665 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; - | ^~~~~~~ -hash.cpp:665:32: warning: unused variable 'subkey5' [-Wunused-variable] - 665 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; - | ^~~~~~~ aligner.cpp: In member function 'int Aligner::check(int)': aligner.cpp:1436:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 1436 | for (i = 0; i < num; i++) { @@ -1237,19 +1403,6 @@ aligner.cpp:1426:7: warning: unused variable 'ret' [-Wunused-variable] 1426 | int ret; | ^~~ -hash.cpp: In member function 'unsigned int HashTable::lookup(int64*, int64*, int, char*, strand, int, HitSet*, bool)': -hash.cpp:746:7: warning: unused variable 'num' [-Wunused-variable] - 746 | int num; - | ^~~ -hash.cpp:750:11: warning: unused variable 'diff' [-Wunused-variable] - 750 | int64 * diff = &tspace1[8], *target = &tspace2[8]; - | ^~~~ -hash.cpp:753:10: warning: unused variable 'sid' [-Wunused-variable] - 753 | uint32 sid, soffset; - | ^~~ -hash.cpp:753:15: warning: unused variable 'soffset' [-Wunused-variable] - 753 | uint32 sid, soffset; - | ^~~~~~~ aligner.cpp: In member function 'AlignRes Aligner::alignSingleEnd(AlignInfo*, char*)': aligner.cpp:1541:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 1541 | for (i = 0; i < nHashTable; i++) { @@ -1257,10 +1410,6 @@ aligner.cpp:1547:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 1547 | for (i = 0; i < res.nRead; i++) { | ~~^~~~~~~~~~~ -main.cpp: In function 'int getArguVal(char*)': -main.cpp:159:15: warning: suggest parentheses around assignment used as truth value [-Wparentheses] - 159 | else if (*c = 'g' || *c == 'G') - | ~~~^~~~~~~~~~~~~~~~~~ aligner.cpp:1629:3: warning: deleting object of abstract class type 'Writer' which has non-virtual destructor will cause undefined behavior [-Wdelete-non-virtual-dtor] 1629 | delete writer; | ^~~~~~~~~~~~~ @@ -1282,42 +1431,10 @@ aligner.cpp:1514:31: warning: unused variable 'strata' [-Wunused-variable] 1514 | bool sorted = info->sorted, strata = info->strata; | ^~~~~~ -hash.cpp: In member function 'unsigned int HashTable::lookupEmbedTable(int64*, uint32, int, char*, strand, int, HitSet*, bool)': -hash.cpp:962:11: warning: unused variable 'noheadkey' [-Wunused-variable] - 962 | int64 * noheadkey = &tspace[8], *rest = &tspace[24]; - | ^~~~~~~~~ -hash.cpp:965:8: warning: unused variable 'subkey4' [-Wunused-variable] - 965 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; - | ^~~~~~~ -hash.cpp:965:32: warning: unused variable 'subkey5' [-Wunused-variable] - 965 | *subkey4 = &tspace[88], *subkey5 = &tspace[104]; - | ^~~~~~~ -hash.cpp:973:11: warning: unused variable 'seqVector' [-Wunused-variable] - 973 | int64 * seqVector = sequence->getSequence(); - | ^~~~~~~~~ aligner.cpp: In member function 'AlignRes Aligner::alignPairEnd(AlignInfo*, char*)': aligner.cpp:1685:17: warning: comparison of integer expressions of different signedness: 'unsigned int' and 'int' [-Wsign-compare] 1685 | for (i = 0; i < res.nRead; i++) { | ~~^~~~~~~~~~~ -hash.cpp: In member function 'bool HashTable::lookup(int64*, uint32)': -hash.cpp:1052:7: warning: unused variable 'num' [-Wunused-variable] - 1052 | int num; - | ^~~ -hash.cpp:1056:11: warning: unused variable 'diff' [-Wunused-variable] - 1056 | int64 * diff = &tspace1[8], *target = &tspace2[8]; - | ^~~~ -hash.cpp:1056:32: warning: unused variable 'target' [-Wunused-variable] - 1056 | int64 * diff = &tspace1[8], *target = &tspace2[8]; - | ^~~~~~ -hash.cpp:1057:11: warning: variable 'seqVector' set but not used [-Wunused-but-set-variable] - 1057 | int64 * seqVector; - | ^~~~~~~~~ -hash.cpp:1059:10: warning: unused variable 'sid' [-Wunused-variable] - 1059 | uint32 sid, soffset; - | ^~~ -hash.cpp:1059:15: warning: unused variable 'soffset' [-Wunused-variable] - 1059 | uint32 sid, soffset; - | ^~~~~~~ aligner.cpp:1754:3: warning: deleting object of abstract class type 'Writer' which has non-virtual destructor will cause undefined behavior [-Wdelete-non-virtual-dtor] 1754 | delete writer; | ^~~~~~~~~~~~~ @@ -1336,14 +1453,6 @@ aligner.cpp:1652:31: warning: unused variable 'strata' [-Wunused-variable] 1652 | bool sorted = info->sorted, strata = info->strata; | ^~~~~~ -hash.cpp: In member function 'uint32 HashTable::printOverflowList(uint32, uint32, int64*)': -hash.cpp:1184:23: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] - 1184 | for (int i = 0; i < keyOffset / BITS_PER_BASE; i++) -hash.cpp:1188:23: warning: comparison of integer expressions of different signedness: 'int' and 'uint32' {aka 'unsigned int'} [-Wsign-compare] - 1188 | for (int i = 0; i < (lenSeq - keyOffset - lenKey) / BITS_PER_BASE; i++) -hash.cpp:1247:27: warning: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'long unsigned int'} [-Wsign-compare] - 1247 | for (int j = 0; j < strlen(set[i].str); j++) - | ~~^~~~~~~~~~~~~~~~~~~~ aligner.cpp: In member function 'AlignRes Aligner::mergeSingleEnd(AlignInfo*, int, char*)': aligner.cpp:1851:3: warning: deleting object of abstract class type 'Writer' which has non-virtual destructor will cause undefined behavior [-Wdelete-non-virtual-dtor] 1851 | delete writer; @@ -1351,19 +1460,6 @@ aligner.cpp:1775:10: warning: unused variable 'ret' [-Wunused-variable] 1775 | int i, ret; | ^~~ -hash.cpp:1254:25: warning: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'long unsigned int'} [-Wsign-compare] - 1254 | for (int j = 0; j < strlen(set[i].str); j++) - | ~~^~~~~~~~~~~~~~~~~~~~ -hash.cpp:1137:11: warning: unused variable 'diff' [-Wunused-variable] - 1137 | int64 * diff = &tspace1[8], *target = &tspace2[8]; - | ^~~~ -main.cpp: In function 'int align(char*, AlignInfo*, char*, int)': -main.cpp:350:7: warning: unused variable 'i' [-Wunused-variable] - 350 | int i, j, ret; - | ^ -main.cpp:350:10: warning: unused variable 'j' [-Wunused-variable] - 350 | int i, j, ret; - | ^ aligner.cpp: In member function 'AlignRes Aligner::mergePairEnd(AlignInfo*, int, char*)': aligner.cpp:1989:3: warning: deleting object of abstract class type 'Writer' which has non-virtual destructor will cause undefined behavior [-Wdelete-non-virtual-dtor] 1989 | delete writer; @@ -1371,49 +1467,6 @@ aligner.cpp:1874:10: warning: unused variable 'ret' [-Wunused-variable] 1874 | int i, ret; | ^~~ -hash.cpp: In member function 'int HashTable::sortList()': -hash.cpp:1430:14: warning: variable 'pool' set but not used [-Wunused-but-set-variable] - 1430 | uint32 * pool; - | ^~~~ -hash.cpp:1321:34: warning: variable 'offset' set but not used [-Wunused-but-set-variable] - 1321 | uint32 entryOffset, seqOffset, offset; - | ^~~~~~ -hash.cpp: In member function 'void HashTable::checkRepeat(uint32)': -hash.cpp:1673:34: warning: variable 'offset' set but not used [-Wunused-but-set-variable] - 1673 | uint32 entryOffset, seqOffset, offset; - | ^~~~~~ -hash.cpp:1690:5: warning: label 'restart' defined but not used [-Wunused-label] - 1690 | restart: if (buckets[bucketId] == HASH_EMPTY - | ^~~~~~~ -main.cpp: In function 'int main(int, char**)': -main.cpp:523:7: warning: unused variable 'numKeys' [-Wunused-variable] - 523 | int numKeys; - | ^~~~~~~ -main.cpp:526:7: warning: unused variable 'nSeq' [-Wunused-variable] - 526 | int nSeq = 1; - | ^~~~ -main.cpp:532:21: warning: unused variable 'sequence' [-Wunused-variable] - 532 | CompactSequence * sequence; - | ^~~~~~~~ -main.cpp:534:15: warning: unused variable 'threadpool' [-Wunused-variable] - 534 | pthread_t * threadpool; - | ^~~~~~~~~~ -main.cpp:536:9: warning: unused variable 'threadid' [-Wunused-variable] - 536 | int * threadid; - | ^~~~~~~~ -In file included from embedhash.cpp:32: -edit_distance.h: At global scope: -edit_distance.h:25:18: warning: 'obj_score_mat' defined but not used [-Wunused-variable] - 25 | static score_mat obj_score_mat = { 1, 1, 1, 1 }; - | ^~~~~~~~~~~~~ -In file included from pair.h:26: -rdtsc.h: In function 'long long unsigned int curtick()': -rdtsc.h:32:10: warning: 'tick' is used uninitialized [-Wuninitialized] - 32 | return tick; - | ^~~~ -rdtsc.h:20:22: note: 'tick' was declared here - 20 | unsigned long long tick; - | ^~~~ builder.cpp: In function 'int getArguVal(char*)': builder.cpp:115:15: warning: suggest parentheses around assignment used as truth value [-Wparentheses] 115 | else if (*c = 'g' || *c == 'G') @@ -1425,40 +1478,16 @@ builder.cpp:277:7: warning: unused variable 'nChar' [-Wunused-variable] 277 | int nChar = sequence->getNum(); | ^~~~~ -hash.cpp: In member function 'unsigned int HashTable::lookupEmbedTable(int64*, uint32, int, char*, strand, int, HitSet*, bool)': -hash.cpp:1049:1: warning: control reaches end of non-void function [-Wreturn-type] - 1049 | } - | ^ -In file included from hash.cpp:31: -edit_distance.h: At global scope: -edit_distance.h:25:18: warning: 'obj_score_mat' defined but not used [-Wunused-variable] - 25 | static score_mat obj_score_mat = { 1, 1, 1, 1 }; - | ^~~~~~~~~~~~~ In file included from main.cpp:38: edit_distance.h: At global scope: edit_distance.h:25:18: warning: 'obj_score_mat' defined but not used [-Wunused-variable] 25 | static score_mat obj_score_mat = { 1, 1, 1, 1 }; | ^~~~~~~~~~~~~ -In file included from pair.h:26: -rdtsc.h: In function 'long long unsigned int curtick()': -rdtsc.h:32:10: warning: 'tick' is used uninitialized [-Wuninitialized] - 32 | return tick; - | ^~~~ -rdtsc.h:20:22: note: 'tick' was declared here - 20 | unsigned long long tick; - | ^~~~ In file included from builder.cpp:34: edit_distance.h: At global scope: edit_distance.h:25:18: warning: 'obj_score_mat' defined but not used [-Wunused-variable] 25 | static score_mat obj_score_mat = { 1, 1, 1, 1 }; | ^~~~~~~~~~~~~ -sequence.cpp: In member function 'int CompactSequence::build(char**, int, int, int)': -sequence.cpp:430:27: warning: 'offsetInOrgSeq' may be used uninitialized [-Wmaybe-uninitialized] - 430 | offsetInOrgSeq++; - | ~~~~~~~~~~~~~~^~ -sequence.cpp:221:33: note: 'offsetInOrgSeq' was declared here - 221 | unsigned int offsetInCmptSeq, offsetInOrgSeq; - | ^~~~~~~~~~~~~~ main.cpp: In function 'void mergeFiles(std::string)': main.cpp:269:21: warning: '%d' directive writing between 1 and 10 bytes into a region of size 8 [-Wformat-overflow=] 269 | sprintf(buf, ".t%d ", i); @@ -1478,6 +1507,13 @@ main.cpp:276:12: note: 'sprintf' output between 4 and 13 bytes into a destination of size 10 276 | sprintf(buf, ".t%d", i); | ~~~~~~~^~~~~~~~~~~~~~~~ +sequence.cpp: In member function 'int CompactSequence::build(char**, int, int, int)': +sequence.cpp:430:27: warning: 'offsetInOrgSeq' may be used uninitialized [-Wmaybe-uninitialized] + 430 | offsetInOrgSeq++; + | ~~~~~~~~~~~~~~^~ +sequence.cpp:221:33: note: 'offsetInOrgSeq' was declared here + 221 | unsigned int offsetInCmptSeq, offsetInOrgSeq; + | ^~~~~~~~~~~~~~ sequence.cpp: In static member function 'static void CompactSequence::compose(char*, int, int64*)': sequence.cpp:676:20: warning: 'code' may be used uninitialized [-Wmaybe-uninitialized] 676 | forward_word = (forward_word << BITS_PER_BASE) | code; @@ -1505,7 +1541,7 @@ g++ -o wham-build sequence.o hash.o embedhash.o builder.o aligner.o interval.o edit_distance.o error.o short.o hitset.o model.o perfcounters.o writer.o -lpthread make[1]: Leaving directory '/build/reproducible-path/wham-align-0.1.5' dh_auto_test - rm -fr -- /tmp/dh-xdg-rundir-HDQy6GMP + rm -fr -- /tmp/dh-xdg-rundir-vlnLfJ20 create-stamp debian/debhelper-build-stamp dh_prep rm -f -- debian/wham-align.substvars @@ -1620,12 +1656,14 @@ dpkg-buildpackage: info: binary-only upload (no source included) dpkg-genchanges: info: not including original source code in upload I: copying local configuration +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/B01_cleanup starting +I: user script /srv/workspace/pbuilder/1650357/tmp/hooks/B01_cleanup finished I: unmounting dev/ptmx filesystem I: unmounting dev/pts filesystem I: unmounting dev/shm filesystem I: unmounting proc filesystem I: unmounting sys filesystem I: cleaning the build env -I: removing directory /srv/workspace/pbuilder/1496476 and its subdirectories -I: Current time: Fri May 30 04:03:16 -12 2025 -I: pbuilder-time-stamp: 1748620996 +I: removing directory /srv/workspace/pbuilder/1650357 and its subdirectories +I: Current time: Sat Apr 27 23:40:53 +14 2024 +I: pbuilder-time-stamp: 1714210853