936913 - Build the editor in unified mode

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[(no longer active)]

This helps with building the code faster, similar to the way that we currently build WebIDL bindings, IPDL protocols, etc.

Try run: https://tbpl.mozilla.org/?tree=Try&rev=94763325e4a1

Comment 1

[(no longer active)]

Attachment: Part 1: Remove the unused editor noisy debug output code

Comment 2

[(no longer active)]

Attachment: Part 2: Build the editor in unified mode

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

What are we losing here? Does this make it harder to debug editor code? Is there a reason to not build all source files in unified mode?

Comment 4

[(no longer active)]

(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #3)
> What are we losing here? Does this make it harder to debug editor code? Is
> there a reason to not build all source files in unified mode?

No, it won't make anything harder since the preprocessor places the correct #line lines so that we don't lose the file name information.  In the general case, you can't mix C or C++ translation units and then get a program with the same meaning as compiling them individually and linking them together, because of things such as naming clashes, static functions/data, etc.  (See the removal of gNoisy in the part 1 patch.)

The other trade-off is that the more you mix translation units, the higher the chance that the unified source file will require rebuilding because of a change to a dependency that would only affect one of the original translation units, so mixing lots of files can make incremental builds slower than they have to be, but the build system will unify a maximum of 16 files together, so that won't be much of an issue.

So we can't just build all source files in unified mode without modification and verification, but we should try to move towards that goal gradually.

Comment 5

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e36c53dd1eb0
https://hg.mozilla.org/integration/mozilla-inbound/rev/831f935fc659

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e36c53dd1eb0
https://hg.mozilla.org/mozilla-central/rev/831f935fc659