search

PiRSquared17 / alembic

Automatically exported from code.google.com/p/alembic
Other
0 stars 0 forks source link

Compiler warnings using gcc #145

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The following is based on a checkout of the code on April 4, 2011.

When building with -Wall the following warnings (which we treat as errors) crop 
up:

1) TimeSamplingType.h:199: warning: 'std::ostream& 
Alembic::AbcCoreAbstract::v1::operator<<(std::ostream&, const 
Alembic::AbcCoreAbstract::v1::TimeSamplingType&)' defined but not used
(bypassed with -Wno-unused-function)

2) GeometryScope.h:114: warning: enumeration value 'kUnknownScope' not handled 
in switch
(bypassed with -Wno-switch)

Original issue reported on code.google.com by robpi...@gmail.com on 4 Apr 2011 at 4:54

GoogleCodeExporter commented 9 years ago 
Sorry, to clarify, when I say "building", I mean "building another application 
that _uses_ Alembic", not "building Alembic"

Original comment by robpi...@gmail.com on 4 Apr 2011 at 4:55

GoogleCodeExporter commented 9 years ago 
It's distinctly possible that this has been fixed in my dev clone, ardent-embic 
(and in the millerlucas-dev), and that this will be resolved in the next update.

The second warning, I just quashed 
(http://code.google.com/r/ardent-embic/source/detail?r=0d1642808ab730b4b3a682926
9c0fe54c1de252f).  The first is likely no longer there (see 
http://code.google.com/r/ardent-embic/source/browse/lib/Alembic/AbcCoreAbstract/
TimeSamplingType.h).

Original comment by ard...@gmail.com on 4 Apr 2011 at 10:40

GoogleCodeExporter commented 9 years ago

Original comment by ble...@gmail.com on 7 Jul 2011 at 10:21

GoogleCodeExporter commented 9 years ago 
I started working on this with a little "spare" time today.
I think I can get this done tomorrow.

Original comment by miller.lucas on 20 Jul 2011 at 1:55

GoogleCodeExporter commented 9 years ago 
I've done a bunch of them here:

http://code.google.com/r/millerlucas-dev/source/detail?r=ec54ccb46a168fda5dbdfba
a51ebd87ccf25af7b

Steve is checking the prman plugin and soon to be included arnold procedural.

There could still be some in a few of the utilties that aren't commonly built.

Dropping the Priority to P2 since the ones that will most likely scare new 
people should now be gone.

Original comment by miller.lucas on 20 Jul 2011 at 9:42

GoogleCodeExporter commented 9 years ago 
-Wextra and -Wno-unused-parameters was added here, along with a bunch of fixes 
for the warning that were found:

http://code.google.com/r/millerlucas-dev/source/detail?r=9958cd1eca64b2781493494
2c2e31a0381a3fbe8

Original comment by miller.lucas on 4 Aug 2011 at 7:32

GoogleCodeExporter commented 9 years ago

Original comment by ble...@gmail.com on 24 Aug 2011 at 11:24

GoogleCodeExporter commented 9 years ago 
The issues specifically in this ticket have been addressed.
Complaints about warnings in the future should get a new ticket.

Original comment by miller.lucas on 24 Aug 2011 at 11:29