MADEAPPS / newton-dynamics

Newton Dynamics is an integrated solution for real time simulation of physics environments.
http://www.newtondynamics.com
Other
936 stars 182 forks source link

Compilation warning on header guard dAnimationTwoWayBlend.h #240

Open execomrt opened 3 years ago

execomrt commented 3 years ago

newton-3.14/sdk/dAnimation/dAnimationTwoWayBlend.h:12:9: warning: 'D_ANIMATION_TWO_WAY_h' is used as a header guard here, followed by #define of a different macro [-Wheader-guard] newton-3.14/sdk/dAnimation/dAnimationTwoWayBlend.h:13:9: note: 'D_ANIMATION_EFFECTOR_BLEND_TWO_WAY_h' is defined here; did you mean 'D_ANIMATION_TWO_WAY_h'?

JulioJerez commented 3 years ago

yes, I made a mistake when doing teh animation for 4.00 it was fixed. thank for the warning.

has you try using 4.0 ? it almost has all the functionality of 3.14 and is far more robust.

execomrt commented 3 years ago

Hello, I've branched 3.14 and also resolved some compilation fixes for Android NDK. Check dgTypes.h for example. Android might not have ARM supported and can have SSE for example. I didn't test yet 4.0.

JulioJerez commented 3 years ago

Hello, I've branched 3.14 and also resolved some compilation fixes for Android NDK.

what change did you make?

JulioJerez commented 3 years ago

also try using 4.0 some good things are coming, that you might like.

On Fri, Aug 6, 2021 at 8:36 AM Stephane Denis @.***> wrote:

Hello, I've branched 3.14 and also resolved some compilation fixes for Android NDK. Check dgTypes.h for example. Android might not have ARM supported and can have SSE for example. I didn't test yet 4.0.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MADEAPPS/newton-dynamics/issues/240#issuecomment-894343135, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6EPJCCBDX2RCAFM5TVF4LT3P6RLANCNFSM5BNLGLPA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .