Update of patch #1476 (project freeciv):
Status: In Progress = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?1476
___
Message
Follow-up Comment #7, patch #1476 (project freeciv):
I just noticed the open source program fontconfig uses macros such as
FC_... It probably won't lead to any confusion since theirs is upper case
and ours is lower, but it's good to be aware of it.
Update of patch #1476 (project freeciv):
Status: Ready For Test = In Progress
___
Reply to this item at:
http://gna.org/patch/?1476
___
Message
Follow-up Comment #5, patch #1476 (project freeciv):
You may want to see also PR #10318
http://bugs.freeciv.org/Ticket/Display.html?id=10318
___
Reply to this item at:
http://gna.org/patch/?1476
Follow-up Comment #6, patch #1476 (project freeciv):
Thanks for the comments, some improvements and fixes more...
Use fc_assert* or not use it? Or did I miss something about the
following comment?
You just missed the successive steps to get this patch ;). I let those
classical assertions at
Follow-up Comment #4, patch #1476 (project freeciv):
I send this to patch #1475 but it belongs here ...
* Use fc_assert* or not use it? Or did I miss something about the following
comment?
+ /* Do not use fc_assert* macros here, since they may call my_vsnprintf()
+ * themselves. */
+
URL:
http://gna.org/patch/?1476
Summary: Replace assert() by fc_assert()
Project: Freeciv
Submitted by: pepeto
Submitted on: samedi 20.02.2010 à 16:15
Category: general
Priority: 3 - Low