Watchdog Service

Here is the patch. The question is - how safe is this method. I guess watchdog $message is a powerfull function with no user input checking. I guess it would be better (for me) to keep it a extra module which could be "log_service" and make some custom (safe input) services.

So what do you think about it? Evil patch?

Btw, I'm using logging for real-time flash remote performance and usability tests, if anybody is interested. say hallo.

Just a little update on flash logging. When I start my App, I create a Logger class that records everything of importance (mousemovemnet, fps changes, clicks and some variables to watch). On drupal, I store this information with a node type "flash log". When the user leaves the page (javascript event), I call the "sendFlashLog" message. This works very well and is better than using the overfilled watchdog. For example:
I collect the information of significant changes in framerate, so every time the fps changes > 5 I store the (time=>fps) pair in an array. When the performance log is submitted, I convert this data into json (I am using an adapter pattern) and on drupal site i wrote a formatter that displays the json data in a fancy flash graph (don't know the name of the graph atm, but googlebing will help).

Maybe this information is usefull for someone.

Btw:
For "on site" flash "debugging", I use ExternalInterface.call("console.log",message) n my custom trace function -> prints flash/flex traces to firebug;