Tim Peters wrote:

>> Author: walter.doerwald
>> Date: Sat Apr  1 22:40:23 2006
>> New Revision: 43545
>>
>> Modified:
>>    python/trunk/Doc/lib/libcalendar.tex
>>    python/trunk/Lib/calendar.py
>> Log:
>> Make firstweekday a simple attribute instead
>> of hiding it behind a setter and a getter.
>
> Walter, what's the purpose of this patch?  The first weekday is still an 
> attribute, but instead of being settable and
> gettable via methods, looks like it's now settable and gettable via 
> module-global functions, and only for the single default
> instance of Calendar created by the module.

This is because in 2.4 there where no Calendar objects and firstweekday was 
only setable and getable via module level functions.

> If so, then (a) the functionality of the Calendar class is weaker now,

No really. firstweekday is changeable simply by assigning to the attribute:

import calendar
cal = calendar.Calendar()
cal.firstweekday = 6

The only thing lost is the range check in the setter.

> and in a
> backward-incompatible way;

Yes, this change is backwards-incompatible, but imcompatible to code that has 
been in the repository since Friday, so this
shouldn't be a problem! ;)
> and,
> (b) the new
> module-global firstweekday() and setfirstweekday() functions are a more 
> obscure way to restore the lost functionality for
> just one
> specific instance of a Calendar subclass.

That's because in 2.4 the module level interface was all there was.

> I don't see the attraction to any part of this.

Simple attribute access looks much more Pythonic to me than setters and gettes 
(especially as the attributes of subclasses are
simple attributes).
Or are you talking about the Calendar class itself?

> [...]

Bye,
   Walter Dörwald



_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com

Reply via email to