[anaconda:master 0/3] Using a pointless overrides checker

Anne Mulhern amulhern at redhat.com
Thu Dec 4 13:41:41 UTC 2014





----- Original Message -----
> From: "Chris Lumens" <clumens at redhat.com>
> To: anaconda-patches at lists.fedorahosted.org
> Sent: Wednesday, December 3, 2014 4:29:55 PM
> Subject: Re: [anaconda:master 0/3] Using a pointless overrides checker
> 
> > The first patch is motivated by the results of the pointless overrides
> > checker.
> > The checker identifies somethings as pointless overrides which aren't
> > really
> > meant to be class attributes at all. It seemed sensible to just refactor
> > all
> > that path calculation into a staticmethod.
> > 
> > The other two patches just add the pylint checker and then remove pointless
> > overrides identified. The checker has to be used with some judgement in
> > when there is multiple inheritance, which occurs with TUIObject, so I did
> > have to check a few by hand. The restrictions of the checker are all
> > documented in the comments.
> 
> I definitely like using pylint to catch more crud like this.  It helps
> improve the quality and clarity of the code by getting rid of things
> that, like you said, are pointless.
> 
> I noticed a fairly busy nested loop or two in
> PointlessClassAttributeOverrideChecker,
> which makes me wonder about the speediness.  Do you have any feeling for
> if it's going to run slowly at all?  I guess in the case of just
> checking a file or two in a commit it'd be fine, but what about running
> across the entire source tree?

I'm so unsophisticated that I always pylint the whole deal.

I ran some tests on blivet with the small sample size of 3.

Results were pretty consistent, and I think they mean that usually the analysis
would add about 1/5th of a second to the typical 18 second completion time.

I suspect that something like the same ratio would hold for anaconda, maybe a bit
better.

<-- SNIP -->

> 
> - Chris
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
> 

- mulhern


More information about the anaconda-patches mailing list