h-vetinari added a comment.
I'd also use newlines more liberally since the two-space indentation is pretty
dense already.
================
Comment at: clang/www/make_cxx_dr_status:190-191
+ % (dr.issue, unresolved_status, dr.status)
+ if not avail.startswith('Sup') and not avail.startswith('Dup'):
+ count[avail] = count.get(avail, 0) + 1
else:
----------------
These two lines are the same with the end of the else-branch, i.e. they could
both be lifted out of the if-clause and unified at the end.
================
Comment at: clang/www/make_cxx_dr_status:198-199
+ % (dr.issue, unresolved_status)
if not avail.startswith('Sup') and not avail.startswith('Dup'):
count[avail] = count.get(avail, 0) + 1
----------------
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138901/new/
https://reviews.llvm.org/D138901
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
