[Mailman-Developers] [Branch~mailman-coders/mailman/2.2] Rev 1046: - Changed someold messages for more current meaning.

Mark Sapiro mark at msapiro.net
Sun Mar 15 00:40:32 CET 2009


Barry Warsaw wrote:
>
>On Mar 14, 2009, at 6:22 PM, noreply at launchpad.net wrote:
>>
>> @@ -510,6 +507,12 @@
>>         try:
>>             msgfp = open(pckfile)
>>             msg = cPickle.load(msgfp)
>> +            if data == {}:
>
>This looks a little Pythonically weird; maybe if len(data) == 0 instead?
>


Thanks Barry.

Actually, I was concerned about the possibility that data could be
None, but upon closer inspection, that can't happen, and even if it
did, it wouldn't matter, so just if data: should be good.

-- 
Mark Sapiro <mark at msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan



More information about the Mailman-Developers mailing list