[dev] [commits] Horde branch master updated. 5ec977a63807b9939c88aaae93a581af03f90385
Michael M Slusarz
slusarz at horde.org
Tue Feb 12 18:46:14 UTC 2013
Quoting Michael J Rubinsky <mrubinsk at horde.org>:
> Quoting Michael M Slusarz <slusarz at horde.org>:
>
>> commit 9f0bc70e891c8a8b1796f2ee6ea11e52f316005f
>> Author: Michael M Slusarz <slusarz at horde.org>
>> Date: Tue Feb 12 00:44:07 2013 -0700
>>
>> Abstract main session class so extended null class doesn't need
>> to extend much of anything
>>
>> Even before this change, much of the null class was already duplicative
>> and unncessary
>
> What was duplicative and unnecessary about the Null class _before_
> the base class was changed? AFAICT, most of the changes were
> necessary due to the use of _data in place of _SESSION.
Agreed about _data/_SESSION. Except there was 6 or 7 methods in the
Null class that were byte-for-byte duplicates of the base method and
contained neither of those variables (and at least 1 private member
variable that was a duplicate).
michael
___________________________________
Michael Slusarz [slusarz at horde.org]
More information about the dev
mailing list