Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Enable gcc -Werror=conversion-null for gcc >= 4.5 for comm-central (port bug 778980)

RESOLVED FIXED in Thunderbird 19.0

Status

MailNews Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: aceman, Assigned: aceman)

Tracking

Trunk
Thunderbird 19.0
All
Linux
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Apply this to comm-central if it is useful.

+++ This bug was initially created as a clone of Bug #778980 +++

To make gcc's NULL/nullptr warnings fatal like clang's nullptr errors, we may want to add -Werror=conversion-null to Linux's CFLAGS/CXXFLAGS for gcc >= 4.5. (The -Wconversion-null flag was new in gcc 4.5.)

If a developer overlooks these gcc warnings, then they may not realize their code change will break the OS X's clang build until they push to the tryserver. This is a waste of developer time and tryserver resources.
(Assignee)

Comment 1

5 years ago
Created attachment 668800 [details] [diff] [review]
patch

Here is the patch. Archaeopteryx, please try it on a try server (linux only is enough). I did not get any failures, but I compile with many components disabled.
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
Thunderbird-Try run: https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=408d5cf11a05
Builds are green
(Assignee)

Comment 3

5 years ago
Comment on attachment 668800 [details] [diff] [review]
patch

The build try server was done for Thunderbird only. Can anybody try Seamonkey?
Attachment #668800 - Flags: review?(mbanner)
SeaMonkey Try run: https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=752bf0e94218
(Assignee)

Comment 5

5 years ago
Thanks for the build.

There is a configure error:
configure: error: Unrecognized extension provided to --enable-extensions: venkman.

That does not seem caused by this patch.
Comment on attachment 668800 [details] [diff] [review]
patch

The first SeaMonkey run I got wrong, but the second should work:

https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=1e27f1fd450a
Attachment #668800 - Flags: review?(mbanner) → review+
(and that's r+ assuming that run passes).
(Assignee)

Comment 8

5 years ago
Now an "Internal server error" and another run with "abort: connection ended unexpectedly" ???
If you this happens during |hg pull -u|:
That happens often. The more changesets, the more likely. Just try again.
If they are blue, they'll automatically be re-run. The builds finished off red in stage-package, but that's because SM is expecting to package the extensions that I disabled (rather than having them pulled). So the actual compilations did pass. That's about the best we can do with SM on try server.
(Assignee)

Comment 11

5 years ago
OK, thanks. So I take it that the build works.

If any breakage appears later please point me to it.
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/ccb28f98f40a
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → Thunderbird 19.0
(Assignee)

Comment 13

5 years ago
Ryan, any reason you did not close this bug after check-in?
Nope, just forgot. Thanks.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.