In chapter five of Professional NT Services, Kevin Miller outlined a nice 
general class for writing event log messages.  I would highly recommend, 
for your sanity, changing each LogEvent method to have a distinct name, 
such as:

 LogSimpleEvent() // No strings
 LogEventSingleString()
 LogEventBinary()

 etc,..

The reason why I would do this is that I spent a little while trying to 
figure out why my event log messages weren't being written, and it turned 
out that it was because the LogEvent call I was making was not calling the 
expected version of the method.  Were it not after 2:00 AM I might have 
noticed this right away, but it burned me for about a half hour.
  
 :)