[ scummvm-Patches-3115757 ] OSYSTEM: Basic logger

[SourceForge.net]

Patches item #3115757, was opened at 2010-11-22 19:37
Message generated for change (Tracker Item Submitted) made by lordhoto
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=3115757&group_id=37116

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Johannes Schickel (lordhoto)
Assigned to: Max Horn (fingolfin)
Summary: OSYSTEM: Basic logger

Initial Comment:
This patch is a proof-of-concept implementation of a basic text only logger.

The "main" addition is the class Log (placed in backends/log/log.[cpp,h]). It has some basic features which can be used for different OSystems, i.e. it optionally outputs time information based on OSystem::getMillies. Apart all it can do is output simple start/end text and it automatically starts with the ScummVM version strings.

I also hooked it up with the SDL backend as an example. Currenlty the output file is always "~/scummvm-log" for testing reasons, thus it should only work fine on UNIX-like systems.

Current TODOs:

- Discuss what exact functionallity we need, i.e. is the simple text output enough?
- Document the class and its methods/members.
- Discuss where/how we want to store logs on our SDL targeted platforms (and implement it)
- Currently OSystem_SDL::logMessage will always call the logging method so that the logger will output the time. This is not really nice when debugN is used. We would need to find some nice way of assuring that the time information is only printed *after* a new line on the next call, to have some nice looking output.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=3115757&group_id=37116