Make Quake 3 remove all sequences of ^ followed by a character other
than ^ to match the engine behavior. This is required for Wolfenstein:
Enemy Territory servers as people use any character (such as ',').
Add a new color flag COLOR_QUAKE3_NUMERIC for games that expect only
^[0-9] to be removed.
Add a new color flag COLOR_QUAKE3_ALPHA for ioquake3-based games that
(combined with COLOR_QUAKE3_NUMERIC) expect only ^[0-9a-zA-Z].
See 6d46b4d84e (commitcomment-28224665)
That Unvanquished P000o color code was implemented from a documentation this now gone, and I'm not able to find any implementation of this in both initial commit and latest commit of Dæmon engine, neither in XreaL nor ET:XreaL codebase. So if it ever existed one day, it never left that tree and was deleted before being used. I will just drop it. If someone needs it one day, this code will remain in git history. Unvanquished/Daemon now uses Xonotic notation for RGB color codes.
Note that if that could would have to be reverted because it was found a game was making use of them, that since [pP] is part of [a-zA-Z] and because Unvanquished also support Quake3 color codes, the Quake3 color code filter would have to be moved after the Unvanquished one.
GAMES_TOTAL is equal to UNKNOWN_SERVER and the GAMES_TOTAL name
is ambiguous because people can believe it's the amount of games
including UNKNOWN_SERVER.
Using UNKNOWN_SERVER instead of GAMES_TOTAL makes explicit that
a code like that is listing every games except the unknown one:
for (i = 0; i < UNKNOWN_SERVER; i+)
In the 'while (current)' loop in stat_servers_input_callback(), the
string list usually has a NULL string (input had newline with no text)
at the end of the server entry. However, there was no NULL string for
the error message when qstat command is not found resulting in a crash.
An alternate way to fix this would be to add a newline to error_msg in
start_qstat().
- xqf version is now computed at configure time using git tags
- a tool named tools/do_checkinstall is added to easily build a package
on checkinstall's supported systems
- now use more than one flag to unescape color codes
this way color escaping can be per-game configured
- also rewrite some flag names to unify the wording
- xonotic RGB color codes are now escaped
Hence, it's now possible to enable:
- GAME_COLOR_QUAKE3
- GAME_COLOR_SAVAGE
- GAME_COLOR_UNVANQUISHED
- GAME_COLOR_XONOTIC
* Enable more warnings.
* current_server_filter is unsigned, don't check if less than 0.
* current_server_filter was allowed for 0 (None) but wasn't suppose
to. The filter menu items are just greyed out (logic issue).
* Some callbacks access a varible but do nothing with it.
* PCX skin loading function returns signed char pointer but is used
as unsigned char pointer in src/skin.c.
gtk_file_chooser_set_filename(.., filename=NULL) internally calls
gtk_file_chooser_unselect_all(..) and then
gtk_file_chooser_select_filename(.., filename) which asserts that
filename is not NULL causing a warning.