Dne 5.8.2015 v 17:24 Simo Sorce napsal(a):
On Wed, 2015-08-05 at 08:20 +0200, Jan Cholasta wrote:
Dne 31.7.2015 v 12:46 Simo Sorce napsal(a):
I've been carrying these patches in my tree for a while, I think it is
time to put them in master as they stand on their own.
Patch 530: ACK
Patch 531: ACK
The methods should be static methods:
def setOption(name, value):
Care to explain why ?
@staticmethod is not used anywhere else in that file.
Because the methods do not use any instance or class state. They will of
course work fine even if they are normal methods, but making them static
methods is cleaner.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code