Jump to content

[update] G3D(lite) from 8.0 beta 4 to 8.0 Final


Guest Neo2003

Recommended Posts

Hello,

I updated G3D to 8.0 final since the one currently in core is a beta one.

Many small mistakes are fixed.

Example:

index 3197ea4..6043aea 100644
--- a/dep/include/g3dlite/G3D/Vector3int16.h
+++ b/dep/include/g3dlite/G3D/Vector3int16.h
@@ -80,7 +80,7 @@ public:
    inline Vector3int16& operator+=(const Vector3int16& other) {
        x += other.x;
        y += other.y;
-        z += other.y;
+        z += other.z;
        return *this;
    }

I tested it on VC80 32/64, VC90 32, VC100 32 and also on Linux Ubuntu 32bits.

Please test it too on your Linux, Windows, MacOS X, ... especially on Linux 64.

Here is the patch, as always drop dep/src/g3dlite and dep/include/g3dlite with files inside the archive and overwrite other files in win folder.

http://www.sendspace.com/file/007k2s

Files are all in DOS format, this is because I use git on Windows. They will be anyway in Unix format when pushed.

Neo2003

Link to comment
Share on other sites

Hi,

First of all thankyou very much for your work on the updated ACE.

Interesting that you are updating G3D and fixing issues in said release. Will the updated G3D resolve the ambiguity in Any.cpp, that throws this ugly warning during compilation (certainly with gcc on *nix, but I am sure it occurs on windows too).

../../../../dep/src/g3dlite/Any.cpp:1069:17: warning: passing NULL to non-pointer argument 1 of ˜G3D::Any::Any(int)˜

Here is the link to the ticket I submitted.

https://mangos.lighthouseapp.com/projects/18208/tickets/638-warning-passing-null-to-non-pointer-argument-1-of-g3danyanyint#ticket-638-3

I posted a patch with the ticket to resolve this issue, but it was deemed invalid because it modified the contents of a third party library. If the patch fixes the warning I can't see the problem, what do you think.

Hope this helps

Link to comment
Share on other sites

At least it compiles here on Ubuntu 10.04 AMD64

But i think the removed lines here are required to compile on i386, because that header is only available with gcc when SSE is explicitly enabled from what i remember:

 // SIMM include
-#ifdef __SSE__
#include <xmmintrin.h>
-#endif

In any case, a diff to the vanilla g3d code would be nice, i don't remember if i commented the changes i had to make for mangos.

Unfortunately, the devs don't seem to think any of the features that need additional 3rd party libs should be optional...

Link to comment
Share on other sites

I have problem compile Rev. 10654 in Centos 5.5

configure.ac:385: the top level

configure.ac:39: error: Autoconf version 2.61 or higher is required

configure.ac:39: the top level

autom4te: /usr/bin/m4 failed with exit status: 63

configure.ac:39: error: Autoconf version 2.61 or higher is required

configure.ac:39: the top level

autom4te: /usr/bin/m4 failed with exit status: 63

autoreconf: /usr/bin/autoconf failed with exit status: 63

I have Autoconf version 2.68

[root@goliath mangos_testg8]# autoconf -V

autoconf (GNU Autoconf) 2.68

Copyright © 2010 Free Software Foundation, Inc.

License GPLv3+/Autoconf: GNU GPL version 3 or later

Please help me...

Thanks in advance.

Edit:

Rezolved.

Link to comment
Share on other sites

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use