Adjust our clang-format rules to include spaces after the hash for nested preprocessor directives

RESOLVED FIXED in Firefox 66

Status

RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

(Blocks: 1 bug)

Trunk
mozilla66

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

2 months ago

This bug is filed to implement the proposal discussed in https://groups.google.com/d/topic/mozilla.dev.platform/lBhFfyh-MJE/discussion.

(Assignee)

Comment 3

2 months ago

I've made a try push to make sure this doesn't break any builds: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8270f6d4f8d365fd20ee2d688fcf4bb31ab2163d

Sylvestre, do you mind checking this if you wanted to autoland the patches after reviewing? Thanks!

Flags: needinfo?(sledru)
https://hg.mozilla.org/integration/mozilla-inbound/rev/9916e7d6e32720362fd18ed03f1acff4884b86df
Bug 1521000 - Part 1: Reformat the tree to ensure everything is formatted correctly with clang-format r=sylvestre

https://hg.mozilla.org/integration/mozilla-inbound/rev/5f4630838d46dd81dadb13220a4af0da9e23a619
Bug 1521000 - Part 2: Adjust our clang-format rules to include spaces after the hash for nested preprocessor directives r=sylvestre
(Assignee)

Updated

2 months ago
Flags: needinfo?(sledru)

Comment 5

2 months ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 2 months ago
status-firefox66: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66

Some conflicts, I fixed the remaining issues in bug 1519636
(and added these two patches in the ignore list)

You need to log in before you can comment on or make changes to this bug.