1505949 - layout/style/BorrowedTypeList.h content should remain on a single line

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, defect)

Description

[Sylvestre Ledru [:Sylvestre]]

Otherwise, it fails with

[task 2018-11-08T16:04:43.835Z] 16:04:43     INFO -  cargo:rerun-if-changed=/builds/worker/workspace/build/src/layout/style/ServoBindings.toml
[task 2018-11-08T16:04:43.836Z] 16:04:43     INFO -  thread '<unnamed>' panicked at 'Unrecognized line in BorrowedTypeList.h: 'GECKO_BORROWED_TYPE_MUT(mozilla::AnimationPropertySegment,'', libcore/option.rs:1000:5
[task 2018-11-08T16:04:43.836Z] 16:04:43     INFO -  stack backtrace:
[task 2018-11-08T16:04:43.836Z] 16:04:43     INFO -     0:     0x7f1f668d875e - std::sys::unix::backtrace::tracing::imp::unwind_backtrace::hf5316c67202e23c8
[task 2018-11-08T16:04:43.836Z] 16:04:43     INFO -                                 at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
[task 2018-11-08T16:04:43.837Z] 16:04:43     INFO -     1:     0x7f1f668b1946 - std::sys_common::backtrace::print::h3068ed2a6dc7ac86
[task 2018-11-08T16:04:43.837Z] 16:04:43     INFO -                                 at libstd/sys_common/backtrace.rs:71
[task 2018-11-08T16:04:43.837Z] 16:04:43     INFO -                                 at libstd/sys_common/backtrace.rs:59
[task 2018-11-08T16:04:43.837Z] 16:04:43     INFO -     2:     0x7f1f668af54d - std::panicking::default_hook::{{closure}}::h1b9d29cc09d41897
[task 2018-11-08T16:04:43.837Z] 16:04:43     INFO -                                 at libstd/panicking.rs:211
[task 2018-11-08T16:04:43.838Z] 16:04:43     INFO -     3:     0x7f1f668af2c0 - std::panicking::default_hook::h3a27b204a313406b
[task 2018-11-08T16:04:43.838Z] 16:04:43     INFO -                                 at libstd/panicking.rs:227
[task 2018-11-08T16:04:43.838Z] 16:04:43     INFO -     4:     0x7f1f668afbfc - std::panicking::rust_panic_with_hook::h0e12cb2fc86d00fa
[task 2018-11-08T16:04:43.838Z] 16:04:43     INFO -                                 at libstd/panicking.rs:477
[task 2018-11-08T16:04:43.838Z] 16:04:43     INFO -     5:     0x7f1f668af7b9 - std::panicking::continue_panic_fmt::h141671b29fe0e27d
[task 2018-11-08T16:04:43.839Z] 16:04:43     INFO -                                 at libstd/panicking.rs:391
[task 2018-11-08T16:04:43.839Z] 16:04:43     INFO -     6:     0x7f1f668af6b5 - rust_begin_unwind
[task 2018-11-08T16:04:43.839Z] 16:04:43     INFO -                                 at libstd/panicking.rs:326
[task 2018-11-08T16:04:43.839Z] 16:04:43     INFO -     7:     0x7f1f669285db - core::panicking::panic_fmt::h429a06507aba9228
[task 2018-11-08T16:04:43.839Z] 16:04:43     INFO -                                 at libcore/panicking.rs:77
[task 2018-11-08T16:04:43.840Z] 16:04:43     INFO -     8:     0x7f1f66928671 - core::option::expect_failed::h4c79c3aae6612643
[task 2018-11-08T16:04:43.840Z] 16:04:43     INFO -                                 at libcore/option.rs:1000
[task 2018-11-08T16:04:43.840Z] 16:04:43     INFO -     9:     0x557aaa55a545 - <core::iter::Map<I, F> as core::iter::iterator::Iterator>::next::hef8bb969c2e8a346
[task 2018-11-08T16:04:43.840Z] 16:04:43     INFO -    10:     0x557aaa560512 - build_script_build::build_gecko::bindings::get_types::h2e64e3a229ff2f7c
[task 2018-11-08T16:04:43.840Z] 16:04:43     INFO -    11:     0x557aaa5674e5 - build_script_build::build_gecko::bindings::generate_bindings::h06710a7a4b1dc497
[task 2018-11-08T16:04:43.841Z] 16:04:43     INFO -    12:     0x7f1f668ef469 - __rust_maybe_catch_panic
[task 2018-11-08T16:04:43.841Z] 16:04:43     INFO -                                 at libpanic_unwind/lib.rs:103
[task 2018-11-08T16:04:43.841Z] 16:04:43     INFO -    13:     0x557aaa553e5b - <F as alloc::boxed::FnBox<A>>::call_box::hc78c33641832ec7a
[task 2018-11-08T16:04:43.841Z] 16:04:43     INFO -    14:     0x7f1f668dcc5a - std::sys_common::thread::start_thread::hb77935aee02382da
[task 2018-11-08T16:04:43.841Z] 16:04:43     INFO -                                 at liballoc/boxed.rs:656
[task 2018-11-08T16:04:43.842Z] 16:04:43     INFO -                                 at libstd/sys_common/thread.rs:24
[task 2018-11-08T16:04:43.842Z] 16:04:43     INFO -    15:     0x7f1f668b08d5 - std::sys::unix::thread::Thread::new::thread_start::hd61f7429b3de1d75
[task 2018-11-08T16:04:43.842Z] 16:04:43     INFO -                                 at libstd/sys/unix/thread.rs:90
[task 2018-11-08T16:04:43.842Z] 16:04:43     INFO -    16:     0x7f1f66441b4f - start_thread
[task 2018-11-08T16:04:43.843Z] 16:04:43     INFO -    17:     0x7f1f65f75fbc - clone

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Attachment: Bug 1505949 - layout/style/{BorrowedTypeList.h,ServoArcTypeList.h,ServoBoxedTypeList.h} content should remain on a single line r?heycam

Make sure that clang-format doesn't touch it

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ed063368a801
layout/style/{BorrowedTypeList.h,ServoArcTypeList.h,ServoBoxedTypeList.h}  content should remain on a single line r=heycam

Comment 3

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/ed063368a801

Comment 4

[(no longer active)]

Attachment: ESR patch

[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): Comment additions only, should not change anything in the binary.  Has also been tested in Nightly for a month or so...

String or UUID changes made by this patch: None

Comment 5

[(no longer active)]

bindgen errors out with this patch on ESR:

 0:05.10 --- stderr
 0:05.10 thread '<unnamed>' panicked at 'Unrecognized line in ServoArcTypeList.h: '// clang-format off'', libcore/option.rs:1000:5

We should use .clang-format-ignore instead...

Comment 6

[(no longer active)]

Attachment: ESR patch

[ESR Uplift Approval Request]

If this is not a sec:{high,crit} bug, please state case for ESR consideration: See comment 4 please.

User impact if declined: 

Fix Landed on Version: 

Risk to taking this patch: Low

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

String or UUID changes made by this patch:

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 9030787 [details] [diff] [review]
ESR patch

Review of attachment 9030787 [details] [diff] [review]:
-----------------------------------------------------------------

thanks!

Comment 8

[Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 9030787 [details] [diff] [review]
ESR patch

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

Comment 9

[(no longer active)]

https://hg.mozilla.org/releases/mozilla-esr60/rev/0081b6ac4c21