> 
>    private void readObject(ObjectInputStream ois)
>           throws java.io.IOException, ClassNotFoundException {
>      ois.defaultReadObject();
>      readLevel(ois);
> 
>      // Make sure that location info instance is set.
>      if (locationInfo == null) {
>        locationInfo = LocationInfo.NA_LOCATION_INFO;   <--- 
> this is noteworthy
>      }
>    }

Would it be simpler to make this the default value for a newly constructed
LoggingEvent then, rather than rely on the readObject method, and EVERY
Receiver impl to have to think about this?  I don't like null values, so
making the declaration of the locationInfo property self initialise to
LocationInfo.NA_LOCATION_INFO makes more sense, unless I am misinterpretting
something.

cheers,

Paul Smith

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

Reply via email to