[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-25 Thread pepeto
Update of patch #1476 (project freeciv): Status: In Progress = Ready For Test ___ Reply to this item at: http://gna.org/patch/?1476 ___ Message

[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-25 Thread David Lowe
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.

[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-23 Thread pepeto
Update of patch #1476 (project freeciv): Status: Ready For Test = In Progress ___ Reply to this item at: http://gna.org/patch/?1476 ___ Message

[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-23 Thread Marko Lindqvist
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

[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-23 Thread pepeto
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

[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-22 Thread Matthias Pfafferodt
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. */ +

[Freeciv-Dev] [patch #1476] Replace assert() by fc_assert()

2010-02-20 Thread pepeto
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