Replaces comparisons using !=, <>, and == against None with the corresponding version of is not None and is None.
Replaces <> with !=.
As requested by reviewers, add spaces around operators on lines touched.
Paths
| Differential D67535
[LNT] Python 3 support: Update `<>` and `None` comparisons ClosedPublic Authored by hubert.reinterpretcast on Sep 12 2019, 9:29 PM.
Details Summary Replaces comparisons using !=, <>, and == against None with the corresponding version of is not None and is None. Replaces <> with !=. As requested by reviewers, add spaces around operators on lines touched.
Diff Detail
Event TimelineComment Actions I haven't put a comment for all occurences of the issues I pointed. Please make a scan through the patch for the whitespaces and isinstance suggestions.
Comment Actions BTW given the size of this patch it might be worth splitting into smaller bits (eg. the type changes, the lambda one and the <> one). Comment Actions
I'll split out the lambda and raise changes first.
hubert.reinterpretcast marked 2 inline comments as done. Comment ActionsReduce patch to updating <> and None comparisons
hubert.reinterpretcast retitled this revision from [LNT] Python 3 support: Minor mechanical changes to [LNT] Python 3 support: Update `<>` and `None` comparisons.Sep 14 2019, 12:04 PM hubert.reinterpretcast added a parent revision: D67587: [LNT] Python 3 support: Update type comparisons and type names. Closed by commit rL371945: [LNT] Python 3 support: Update `<>` and `None` comparisons (authored by hubert.reinterpretcast). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 220258 lnt/trunk/lnt/external/stats/pstat.py
lnt/trunk/lnt/external/stats/stats.py
lnt/trunk/lnt/server/ui/filters.py
lnt/trunk/tests/server/db/ImportV4TestSuiteInstance.py
|