Opened 5 years ago

Last modified 5 years ago

#5505 new defect

FCollada build warnings on clang / Apple LLVM 10.0.1

Reported by: historic_bruno Owned by:
Priority: Should Have Milestone: Backlog
Component: Build & Packages Keywords: mac
Cc: Krinkle Patch:

Description

See also #5499.

  • macOS Mojave 10.14.5
  • Apple LLVM version 10.0.1 (clang-1001.0.46.3)
  • Xcode 10.2 (10E125)

[-Wdeprecated-declarations]

FCollada/FUtils/FUAssert.cpp:54:3: warning: 'Debugger' is deprecated: first
      deprecated in macOS 10.8 [-Wdeprecated-declarations]
                Debugger();
                ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/MacTypes.h:708:1: note: 
      'Debugger' has been explicitly marked deprecated here
Debugger(void)                                                __OSX_AVAI...
^

[-Wdynamic-class-memaccess]

In file included from FCollada/FCDocument/FCDAnimated.cpp:15:
In file included from FCollada/StdAfx.h:13:
In file included from FCollada/FCollada.h:49:
In file included from FCollada/FUtils/FUtils.h:81:
In file included from FCollada/FMath/FMath.h:38:
FCollada/FMath/FMArray.h:300:14: warning: destination for this 'memcpy' call is
      a pointer to dynamic class 'FUTrackedList<FCDAnimationCurve>'; vtable
      pointer will be overwritten [-Wdynamic-class-memaccess]
                                                memcpy(newValues, heapBu...
                                                ~~~~~~ ^
FCollada/FMath/FMArray.h:229:4: note: in instantiation of member function
      'fm::vector<FUTrackedList<FCDAnimationCurve>, false>::reserve' requested
      here
                        reserve(count);
                        ^
FCollada/FCDocument/FCDAnimated.cpp:38:9: note: in instantiation of member
      function 'fm::vector<FUTrackedList<FCDAnimationCurve>, false>::resize'
      requested here
        curves.resize(valueCount);
               ^
FCollada/FMath/FMArray.h:300:14: note: explicitly cast the pointer to silence
      this warning
                                                memcpy(newValues, heapBu...
                                                       ^
                                                       (void*)

[-Wunused-private-field]

In file included from FCollada/FCDocument/FCDExtra.cpp:12:
FCollada/FCDocument/FCDExtra.h:444:12: warning: private field 'parent' is not
      used [-Wunused-private-field]
        FCDEType* parent;
                  ^

[-Wundefined-var-template]

In file included from FCollada/FCDocument/FCDocument.cpp:34:
FCollada/FCDocument/FCDLibrary.h:53:2: warning: instantiation of variable
      'FCDLibrary<FCDAnimation>::__classType' required here, but no definition
      is available [-Wundefined-var-template]
        DeclareObjectType(FCDObject);
        ^
FCollada/FUtils/FUObjectType.h:92:61: note: expanded from macro
      'DeclareObjectType'
        virtual const FUObjectType& GetObjectType() const { return __classType; } \

[-Wself-assign-field]

In file included from FCollada/FColladaPlugin.cpp:9:
In file included from FCollada/StdAfx.h:13:
In file included from FCollada/FCollada.h:49:
In file included from FCollada/FUtils/FUtils.h:115:
FCollada/FUtils/FUTracker.h:122:7: warning: assigning field to itself
      [-Wself-assign-field]
                ptr = ptr;
                    ^

[-Wself-assign-overloaded]

FCollada/FCDocument/FCDPhysicsForceFieldInstance.cpp
FCollada/FCDocument/FCDParameterAnimatable.cpp:308:5: warning: explicitly
      assigning value of variable of type 'FCDParameterListAnimatableT<T, Q>' to
      itself [-Wself-assign-overloaded]
        v1 = v1;
        ~~ ^ ~~

[-Woverloaded-virtual]

In file included from FCollada/FCDocument/FCDPhysicsRigidBody.cpp:12:
In file included from FCollada/FCDocument/FCDPhysicsShape.h:21:
FCollada/FCDocument/FCDPhysicsAnalyticalGeometry.h:322:40: warning: 
      'FCDPASTaperedCapsule::Clone' hides overloaded virtual function
      [-Woverloaded-virtual]
        virtual FCDPhysicsAnalyticalGeometry* Clone(FCDPhysicsAnalytical...
                                              ^
FCollada/FCDocument/FCDPhysicsAnalyticalGeometry.h:280:21: note: hidden
      overloaded virtual function 'FCDPASCapsule::Clone' declared here: type
      mismatch at 1st parameter ('FCDEntity *' vs 'FCDPhysicsAnalyticalGeometry
      *')
        virtual FCDEntity* Clone(FCDEntity* clone = NULL, bool cloneChil...
                           ^

[-Wfor-loop-analysis]

FColladaPlugins/FArchiveXML/FArchiveXML.cpp:404:2: warning: variable 'x' used in
      loop condition not modified in loop body [-Wfor-loop-analysis]
        _FCATCH_ALL
        ^~~~~~~~~~~
FCollada/FCollada.h:39:37: note: expanded from macro '_FCATCH_ALL'
#define _FCATCH_ALL for (int x = 0; x != 0;)
                                    ^

Attachments (1)

macos-fcollada-buildwarnings.txt (250.8 KB ) - added by historic_bruno 5 years ago.

Download all attachments as: .zip

Change History (4)

by historic_bruno, 5 years ago

comment:1 by Krinkle, 5 years ago

Owner: set to Krinkle
Status: newassigned

comment:2 by historic_bruno, 5 years ago

Note: sometimes we silence the warnings when they are harmless, sometimes a fix is required if it might truly break FCollada in the future. If choosing to silence the warnings, it's necessary to verify that the flags are supported on the oldest gcc/clang we support (see CppSupport).

comment:3 by Krinkle, 5 years ago

Cc: Krinkle added
Owner: Krinkle removed
Status: assignednew
Note: See TracTickets for help on using tickets.