For some reason, this old Coverity issue was already marked as solved.
Also it is slightly more effective to store the end pointer than
evaluate in every pass.
fixed the same commit 43ba6b4e as before
The reference was used instead of iterator, but the reference cannot be reused because the second assignment overrides the original node contents instead of working on the second node.
fixes warnings with Xcode 14.3 as in previous commit
+ allocate +1 in Log_output buffer - snprintf(MULL, 0...) returns number
of bytes without terminating '\0'. std::string storage isn't guaranteed
to be NULL-terminated (std::string("").at(0) throws an exception). The
buffer is somewhere preallocated 256 which prevents problem but it's
better not to rely upon it.
+ comment out some unused function in LDGM that also triggered the
warning but looked a bit suspicious so it was not clear how to fix
nvcc had some problems with gcc 12.1 stl map templates when compiling gpu.cu.
And since ldgm-session-gpu.h which contained the c++ code was only needed for
this one macro, it's probably better to move it to gpu.cuh which is
already included by all files that need it.
Doesn't differ from main.cpp much and it is in history, anyways.
This change is mostly to allow batch removal of .orig files in repo
without affecting a tracked file.
This reduces sources size by a great deal - matrices occupied some
33 MBs.
Seems useless - matrices are should be perhaps reconstructed by
matrix-gen but seem incompatible with current version (doesn't
have k/m/c header).
The scripts are not compatible with current version (and used data/
is missing).
