QC issue, too many warnings, r11598 trunk
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Inkscape |
Triaged
|
Low
|
Unassigned | ||
Bug Description
When I built r11598 (trunk) on Ubuntu 12.04 today there were 182 warnings emitted. That is far too many. Barring the very rare case where there is just no way to work around a compiler warning, this number would ideally be zero. Yes, most of them are probably harmless, but until proven otherwise, one would have to assume that every one of them is a bug. This is why it is just bad form to have this many in trunk.
The warnings broken down by gcc -W* error are:
6 deprecated-
1 format
8 missing-
12 sign-compare
6 strict-aliasing
4 switch
2 type-limits
15 uninitialized
119 unused-
9 write-strings
The list is from:
make 2>&1 | tee build_1.log
grep 'warning:' build_1.log > warnings.txt
I understand that warnings can show up on one platform that were not present on another. But these should have shown up on either Linux or Mac, which is where most of the development occurs.
| David Mathog (mathog) wrote | #1 |
| su_v (suv-lp) wrote | #3 |
| Changed in inkscape: | |
| importance: | Undecided → Low |
| status: | New → Triaged |
| jazzynico (jazzynico) wrote | #4 |
| David Mathog (mathog) wrote | #5 |
| tags: | added: code-design |
Warnings about breaking strict aliasing rules:
Bug #168548 ”inkscape-0.45.1 breaks strict aliasing rules”
Bug #323414 "strict-aliasing rules warnings”
Warnings about deprecated functions in Pango are tracked in
Bug #166276 “pango_
(see also Bug #1016020)
Warnings from poppler include files -> upstream issue (poppler -> Xpdf)
<http://
(possibly that policy has changed by now)
Warnings from local libgdl:
Migration to use external shared lib is on the roadmap, tracked in
Bug #792115 “Build against external libgdl”
Warnings from local libcroco:
Migration to use external shared lib is on the roadmap, tracked in
Bug # #648246 "Get rid of embedded copy of libcroco”
(Note: earlier reports about libcroco warnings had been closed as 'Won't fix': e.g. bug #169400, bug #169401)