[clang] [clang] Do less advertising for unresolved issues in `cxx_dr_status.html` (PR #78836)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 12 23:59:10 PST 2024
================
@@ -159,21 +165,37 @@ def availability(issue):
if status == 'unknown':
avail = 'Unknown'
avail_style = ' class="unknown"'
- elif re.match('^[0-9]+\.?[0-9]*', status):
- avail = 'Clang %s' % status
- if float(status) > latest_release:
- avail_style = ' class="unreleased"'
- else:
- avail_style = ' class="full"'
+ elif re.match(r'^[0-9]+\.?[0-9]*', status):
+ if not proposed_resolution:
+ avail = 'Clang %s' % status
+ if float(status) > latest_release:
+ avail_style = ' class="unreleased"'
+ else:
+ avail_style = ' class="full"'
+ else:
+ avail = 'Not Resolved*'
+ avail_style = f' title="Clang {status} implements {proposed_resolution} resolution"'
elif status == 'yes':
- avail = 'Yes'
- avail_style = ' class="full"'
+ if not proposed_resolution:
+ avail = 'Yes'
+ avail_style = ' class="full"'
+ else:
+ avail = 'Not Resolved*'
+ avail_style = f' title="Clang implements {proposed_resolution} resolution"'
elif status == 'partial':
- avail = 'Partial'
- avail_style = ' class="partial"'
+ if not proposed_resolution:
+ avail = 'Partial'
+ avail_style = ' class="partial"'
+ else:
+ avail = 'Not Resolved*'
+ avail_style = f' title="Clang partially implements {proposed_resolution} resolution"'
elif status == 'no':
- avail = 'No'
- avail_style = ' class="none"'
+ if not proposed_resolution:
+ avail = 'No'
+ avail_style = ' class="none"'
+ else:
+ avail = 'Not Resolved*'
+ avail_style = f' title="Clang does not implement {proposed_resolution} resolution"'
----------------
cor3ntin wrote:
In the no case, I would leave it unknown / no comment... that we don't do something is not useful information for users. WDYT?
https://github.com/llvm/llvm-project/pull/78836
More information about the cfe-commits
mailing list