[Gnash-commit] [SCM] Gnash branch, master, updated. release_0_8_9_final-1464-g941e9a6

[Bastiaan Jacques]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Gnash".

The branch, master has been updated
       via  941e9a6a32e7394c76ceeba84ad5eed03b0c98cd (commit)
       via  c33df3e23c3360c71328356e20e327de7d415885 (commit)
       via  352acff6d8b228f67e77fa1eb0d3d3bb85de3b33 (commit)
      from  55ea6acde7fdad76449627cccb513cea7069151a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.savannah.gnu.org/cgit//commit/?id=941e9a6a32e7394c76ceeba84ad5eed03b0c98cd


commit 941e9a6a32e7394c76ceeba84ad5eed03b0c98cd
Author: Bastiaan Jacques <address@hidden>
Date:   Sun Apr 29 15:53:16 2012 +0200

    Fixing access to a private member; even though GCC accepts this
    (See GCC #41437), Clang does not.

diff --git a/libcore/swf/DefineButtonTag.h b/libcore/swf/DefineButtonTag.h
index 7e1281c..9910e06 100644
--- a/libcore/swf/DefineButtonTag.h
+++ b/libcore/swf/DefineButtonTag.h
@@ -157,8 +157,6 @@ public:
         return (_conditions & KEYPRESS);
     }
 
-private:
-
     /// Return the keycode triggering this action
     //
     /// Return 0 if no key is supposed to trigger us
@@ -166,6 +164,8 @@ private:
         return (_conditions & KEYPRESS) >> 9;
     }
 
+private:
+
     enum Condition
     {
         IDLE_TO_OVER_UP = 1 << 0,

http://git.savannah.gnu.org/cgit//commit/?id=c33df3e23c3360c71328356e20e327de7d415885


commit c33df3e23c3360c71328356e20e327de7d415885
Author: Bastiaan Jacques <address@hidden>
Date:   Sun Apr 29 15:52:40 2012 +0200

    Clang dislikes the assignment to itself.

diff --git a/libcore/vm/VM.h b/libcore/vm/VM.h
index af51c93..4bab9b9 100644
--- a/libcore/vm/VM.h
+++ b/libcore/vm/VM.h
@@ -292,7 +292,7 @@ private:
 inline ObjectURI
 getURI(const VM& vm, const std::string& str, bool lowerCaseHint=false)
 {
-    lowerCaseHint=lowerCaseHint; // TODO pass hint to ObjectURI ctor
+    UNUSED(lowerCaseHint); // TODO pass hint to ObjectURI ctor
     // Possible optimization here is to directly compute
     // noCase value if VM version is < 7
     return ObjectURI((NSV::NamedStrings)vm.getStringTable().find(str));

http://git.savannah.gnu.org/cgit//commit/?id=352acff6d8b228f67e77fa1eb0d3d3bb85de3b33


commit 352acff6d8b228f67e77fa1eb0d3d3bb85de3b33
Author: Bastiaan Jacques <address@hidden>
Date:   Sun Apr 29 15:51:36 2012 +0200

    Clang refuses to compile the npapi version test; changing the main()
    signature fixes that.

diff --git a/macros/npapi.m4 b/macros/npapi.m4
index e3bde2f..7356fb6 100644
--- a/macros/npapi.m4
+++ b/macros/npapi.m4
@@ -89,7 +89,7 @@ AC_DEFUN([GNASH_PATH_NPAPI],
     AC_COMPILE_IFELSE([AC_LANG_SOURCE([
                   #include "npapi.h" 
                   #include "npruntime.h"
-                  int main(int argc, char* argv[]){
+                  int main(int argc, char** argv){
                    NPString str;
                     uint32_t len = str.UTF8Length;
                    return 0;

-----------------------------------------------------------------------

Summary of changes:
 libcore/swf/DefineButtonTag.h |    4 ++--
 libcore/vm/VM.h               |    2 +-
 macros/npapi.m4               |    2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)


hooks/post-receive
-- 
Gnash