URL:
  <http://gna.org/patch/?1404>

                 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

    _______________________________________________________

Details:

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
--enable-debug.
* 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

<http://gna.org/patch/download.php?file_id=7650>

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?1404>

_______________________________________________
  Message posté via/par Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to