Commit Briefs


Neels Hofmeyr

reflect ignore-whitespace in atom hash


Neels Hofmeyr

patience: use qsort




Neels Hofmeyr

debug: patience: mark unique debug



Neels Hofmeyr

debug fix: missing space



Stefan Sperling

fix build with PROFILE=1




Neels Hofmeyr

add test019 showing max-effort bug


Neels Hofmeyr

debug output fixes

- fix compilation error with DEBUG 1, after moving patience state out of diff_atom, and after renaming diff_data.d to diff_data.root. - add some missing newlines. - also debug "HIT" on a max-effort cut, like when finding a mid snake, which eases finding all the places where boxes get cut.




Stefan Sperling

fix off-by-one access beyond mapped file in diff_data_atomize_text_lines_mmap()

Thread 1 received signal SIGSEGV, Segmentation fault. 0x0000013992a89eca in diff_data_atomize_text_lines_mmap (d=0x13b9b455668) \ at /home/stsp/src/got/got/../lib/diff_atomize_text.c:134 134 if (line_end[0] == '\r' (gdb) p pos $1 = (const uint8_t *) 0x13be402006d "" (gdb) p end $2 = (const uint8_t *) 0x13be4023000 <error: Cannot access memory at \ address 0x13be4023000> (gdb) p end-1 $3 = (const uint8_t *) 0x13be4022fff "" (gdb) p line_end $4 = (const uint8_t *) 0x13be4023000 <error: Cannot access memory at \ address 0x13be4023000>









Stefan Sperling

fix another format string issue in results_test.c

Cast ARRAYLIST_IDX result to unsigned long and use %lu in the format string. Clang 10.0.1 complained as follows: results_test.c:148:9: warning: format specifies type 'int' but the argument has type 'long' [-Wformat] ARRAYLIST_IDX(c, result->chunks), ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~