[pkg-apparmor] UDD notifications script

u u at 451f.org
Mon Mar 9 12:42:16 UTC 2015


Hi,

Holger Levsen:
> On Sonntag, 1. März 2015, Christian Boltz wrote:
>>> instead write:
>>>     return [ {'id': item[0], 'tag': item[1], 'title': item[2]}
>>>              for item in cursor.fetchall() ]
>> More readable and understandable? I slightly ;-) doubt
> 
> /me agrees. (also with intrigeri's reply ;)

Thank you all for the review!

I like the Python style, but I have indeed to admit that I also find it
less readable. I optimized the temporary variables though and applied it
in one function. I kept my initial version in the construction of the
added_usertags and deleted_usertags, where I found this way of writing
much too hard to read.

I still did not import the script to a common repository on Alioth.
I requested access to the team though, so that I would share a common
group access with you. intrigeri, could you check if you received my
request and approve it if you think that that's the way to go?

Cheers!
u.



More information about the pkg-apparmor-team mailing list