<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 3, 2015 at 3:41 AM, Christian Boltz <span dir="ltr"><<a href="mailto:apparmor@cboltz.de" target="_blank">apparmor@cboltz.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
Am Montag, 3. August 2015 schrieb Kshitij Gupta:<br>
> On Sat, Jul 18, 2015 at 2:23 AM, Christian Boltz wrote:<br>
...<br>
> > [ 77-split-logparser-add_event_to_tree.diff ]<br>
<span class=""><br>
> > --- utils/apparmor/logparser.py 2015-07-17 22:43:21.977879320 +0200<br>
> > +++ ./utils/apparmor/logparser.py       2015-07-17<br>
</span>...<br>
<span class="">> > +    def map_log_type(self, type):<br>
><br>
> "type" is a keyword in Python so using it/overriding it in this scope<br>
> is not a good idea and we can probably avoid it. How about "log_type"<br>
> instead?<br>
><br>
> I'm aware that "type" has been incorrectly used as a variable at other<br>
> places, but two wrongs don't make a right ;-)<br>
<br>
</span>Good point. I'll change it to log_type before commiting (and wonder why<br>
python doesn't complain about redefining keywords ;-)<br>
<br></blockquote><div>pylint probably does complain about redefining keywords I think.<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
<br>
Christian Boltz<br>
<span class="HOEnZb"><font color="#888888">--<br>
Lass es mich so sagen: GUIs? Wir haben keine. Davon aber zwei.<br>
[Ratti in suse-linux]<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
--<br>
AppArmor mailing list<br>
<a href="mailto:AppArmor@lists.ubuntu.com">AppArmor@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/apparmor" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/apparmor</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div>Regards,<br><br></div>Kshitij Gupta<br></div></div>
</div></div>