#32921: Code and script changes to run clang-format without breaking checkSpaces or coccinelle ----------------------------+------------------------------------ Reporter: nickm | Owner: nickm Type: enhancement | Status: needs_revision Priority: Medium | Milestone: Tor: 0.4.3.x-final Component: Core Tor/Tor | Version: Severity: Normal | Resolution: Keywords: style, 043-can | Actual Points: 1.5 Parent ID: #29226 | Points: Reviewer: catalyst | Sponsor: ----------------------------+------------------------------------ Changes (by catalyst):
* status: needs_review => needs_revision Comment: Following up from last week's C style meeting: I think this is almost ready, given what we agreed on as the scope of this ticket. There should be disclaimers in the .clang-format and the script about how the style isn't official yet, and that people shouldn't commit or merge the results yet. Maybe also a comment about the minimum required clang- format version, which I think is 6.0 but might be earlier? We should open a separate ticket to parameterize the clang-format executable name to make it easier for developers to run the minimum required clang-format version even when it's not called clang-format (e.g., clang-format-6.0 on Xenial). nickm, would you like to do that? I can also open it if you like. -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/32921#comment:23> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs