Session.enabled is accessed in both synchronized and unsynchronized contexts.

Access to Session.enabled is synchronized on Session everywhere except next() in the trunk. Is this intended?

In 1.1.x branch it appears that next() itself is synchronized.

I also notice that there's a synchronized (this) in Session's constructor. I think this is redundant?