Dave - how about a new (simple) inspection for classes that define a
logger, but don't include any logging statements ( effectively making
the logging pointless )?
Its pretty the same as a "unused member" which would probably be picked
up else where, but I was just picturing it as being usefull to be
context specific here...