Summary: log module
                 Project: Freeciv
            Submitted by: pepeto
            Submitted on: dimanche 10.01.2010 à 16:32
                Category: general
                Priority: 1 - Later
                  Status: Ready For Test
                 Privacy: Public
             Assigned to: pepeto
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 2.3.0



Some observations in the current implementation:
* freelog() messages always compute all arguments, even if it won't be
displayed to the screen.
* The --debug option is not flexible, it overwrite the previous one.
* "4:file" doesn't work if the level is different of LOG_DEBUG.

What this patch changes:
* freelog(loglevel) renamed log_fatal(), log_error(), log_normal(),
log_verbose() and log_debug().
* Those macros don't compute the arguments if they won't display them.
* log_debug() really doesn't have any effect when not configured with
* The --debug allows many levels based on the location of the call of the log
macro (file + line) when configured with --enable-debug.
* RETURN[_VAL]_IF_FAIL[_MSG]() renamed log_assert[_ret][_val][_msg]().


File Attachments:

Date: dimanche 10.01.2010 à 16:32  Name: trunk_log.diff.bz2  Size: 116 ko  
By: pepeto



Reply to this item at:


  Message posté via/par Gna!

Freeciv-dev mailing list

Reply via email to