Hi Eugene,
On 2026-09-14 20:15:32, you wrote to me:
All three are cosmetic and pre-date the unicode branch. In gs_max3.h
the packed attribute sits after the typedef name, so it applies to the name and every compiler drops it (gcc 15 and clang say the same). The
enum is held at 16 bits by the rsvd=32766 member anyway.
The two in gs_wtr.h are the same thing twice: GroupDescRecord and UserBaseRecord are variables of anonymous structs rather than
typedefs, which gcc 4.8 on Leap 42.1 reports as a type without linkage
and gcc 5 and later no longer do. None of them changes the binary.
Ok, thanks for the clarification.
Yesterday I also tried compiling on a more modern system with gcc 15.3.0.. The gs_wtr.h warnings were indeed gone, but I got this extra one. (I don't know if this one is from before the unicode branch)
[ 95%] Building CXX object golded3/CMakeFiles/golded.dir/geusrbse.cpp.o ~/Projects/golded-plus-utf8/golded3/geusrbse.cpp: In member function 'virtual void guserbase::print_line(uint, uint, bool)':
~/Projects/golded-plus-utf8/golded3/geusrbse.cpp:239:48: warning: ignoring return value of 'std::__cxx11::basic_string<_Char
T, _Traits, _Alloc> std::operator+(__cxx11::basic_string<_CharT, _Traits, _Alloc>&&, const _CharT*) [with _CharT = char; _Traits = char_traits<char>; _Alloc = allocator<char>]', declared with attribute 'nodiscard' [-Wunused-result]
239 | ((useraddr = "<") + entry.iaddr) + ">";
| ^~~
In file included from /usr/include/c++/15/string:56,
from /usr/include/c++/15/bits/locale_classes.h:42,
from /usr/include/c++/15/bits/ios_base.h:43,
from /usr/include/c++/15/ios:46,
from /usr/include/c++/15/bits/ostream.h:43,
from /usr/include/c++/15/ostream:42,
from /usr/include/c++/15/iostream:43,
from ~/Projects/golded-plus-utf8/golded3/golded.h:35,
from ~/Projects/golded-plus-utf8/golded3/geusrbse.cpp:29: /usr/include/c++/15/bits/basic_string.h:3973:5: note: declared here
3973 | operator+(basic_string<_CharT, _Traits, _Alloc>&& __lhs,
| ^~~~~~~~
Bye, Wilfred.
--- FMail-lnx64 2.3.4.1-B20260520
* Origin: FMail development HQ (2:280/464)