> So I'm close to just disallowing any explicit dependencies. 

Do you see any possiblity of dependencies that aren't loggers?

So, instead of 

  LoggerAspect loggerAspect = new LoggerAspectImpl( something );
  kernel.addAspect( loggerAspect );

  SecurityAspect secAspect = new SecurityAspectImpl( loggerAspect );
  kernel.addAspect( secAspect );

We can have:

  LoggerAspect loggerAspect = new LoggerAspectImpl( something );
  kernel.addAspect( loggerAspect );

  SecurityAspect secAspect = new SecurityAspectImpl(
loggerAspect.getRootLogger() );
  kernel.addAspect( secAspect );

                             ...

public class SecurityAspectImpl implements SecurityAspect
{
  public SecurityAspectImpl()
  {
    this( new NullLogger() );
  }

  public SecurityAspectImpl( Logger logger )
  {
    m_logger = logger;
  }
}

                             *or*


public class SecurityAspectImpl extends AbstractLogEnabled implements
SecurityAspect
{
  public SecurityAspectImpl()
  {
    this( new NullLogger() );
  }

  public SecurityAspectImpl( Logger logger )
  {
    enableLogging( logger );
  }
}


--
hammett



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to