clang-format NSPR
Categories
(NSPR :: NSPR, enhancement)
Tracking
(Not tracked)
People
(Reporter: KaiE, Assigned: KaiE)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 1 obsolete file)
9.21 KB,
text/plain
|
Details | |
3.92 KB,
patch
|
Details | Diff | Splinter Review |
It would be nice to reformat the NSPR code, like we did with NSS.
Assignee | ||
Comment 1•5 years ago
|
||
I'd like to do this soon. But before that, we need to complete bug 1535665, to ensure Ludo's patches don't bitrot.
Assignee | ||
Comment 2•5 years ago
|
||
Mike, I'd like to reformat NSPR in the same way as we do for NSS, same rules, same version of clang-format.
Are you OK with that? Can I use your r+ for that change, or maybe I don't need a review at all?
I've already tested the reformatted code with both NSS test suite and Firefox, and didn't see any regressions.
I'm attaching the list of files that will be touched.
Assignee | ||
Comment 3•5 years ago
|
||
This is the copied .clang-format file from NSS, and a copied and modified version of the run_clang_format.sh file.
The only modification to run_clang_format.sh is the list of excluded directories.
This script won't be used automatically, but can executed on demand.
In a second step we can consider to hook it up to the NSS taskcluster automation.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 5•5 years ago
•
|
||
Mike, I had sent you an email with subject "nspr-reformat.patch", did you get it?
It's pretty large, I didn't want to attach it in bugzilla.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 7•5 years ago
|
||
Assignee | ||
Comment 8•5 years ago
|
||
Done. I guess it's OK for skimming and having a look. But reviewing the individual changes would be overkill.
The goal is that we'd not touch the formatting ourselves, but rely on clang-format to decide how it looks. The same is done for NSS.
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 9•4 years ago
|
||
This is more work than I thought. I'd like to start with bug 1581890 instead.
Assignee | ||
Comment 10•4 years ago
|
||
I'm not happy how clang-format behaves with NSPR.
I like the artistic style utility better, because it's less invasive.
Given that NSPR isn't developed very actively, it might be sufficient to a one-time cleanup, and only occasionally re-run a prettier.
Therefore I'd like to close this for now, and only do the work in bug 1581890.
Description
•