1510535 - Restore comment in .clang-format-ignore

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

Bug 1510128 removed a comment that was pretty helpful IMO.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Bug 1510535 - Restore a few lines in .clang-format-ignore. r?tcampbell!,sylvestre!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2449004c31ff
Restore a few lines in .clang-format-ignore. r=tcampbell,sylvestre

Comment 3

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/2449004c31ff

Comment 4

[(no longer active)]

Comment on attachment 9028208 [details]
Bug 1510535 - Restore a few lines in .clang-format-ignore. r?tcampbell!,sylvestre!

[ESR Uplift Approval Request]

If this is not a sec:{high,crit} bug, please state case for ESR consideration: This is required for easier backporting of patches after the reformatting of ESR using clang-format.

User impact if declined: Declining this will negatively impact our developers' ability to easily backport their patches to ESR in the future.

Fix Landed on Version: 65

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): This is NPOTB

String or UUID changes made by this patch: None

Comment 5

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 9028208 [details]
Bug 1510535 - Restore a few lines in .clang-format-ignore. r?tcampbell!,sylvestre!

OK for uplift to ESR60 as part of the clang-format project.

Comment 6

[(no longer active)]

https://hg.mozilla.org/releases/mozilla-esr60/rev/2d177480ed27