# HG changeset patch
# User Michael Goffioul <michael.goffioul@gmail.com>
# Date 1319120378 -3600
# Node ID deb69dab45148c9e3ee9a6644ba72b9b2019ecd0
# Parent  2cd903770748c27a53035428cc93400cfa084d31
Avoid some compilation warnings.

* graphics.cc (base_properties::is_handle_visible): Add parenthesis.
(uicontrol::properties::get_extent): Don't use "parent" variable.
(uicontrol::properties::update_units): Likewise.
(uipanel::properties::update_units): Likewise.
(gh_manager::do_event_processing): Rename "lock" variable into "guard".
* ov.h (DEF_DUMMY_VALUE_EXTRACTOR): Remove unused argument name.

diff -r 2cd903770748 -r deb69dab4514 src/graphics.cc
--- a/src/graphics.cc	Thu Oct 20 14:08:25 2011 +0100
+++ b/src/graphics.cc	Thu Oct 20 15:19:38 2011 +0100
@@ -2542,7 +2542,7 @@
 base_properties::is_handle_visible (void) const
 {
   return (handlevisibility.is ("on")
-          || executing_callback && ! handlevisibility.is ("off"));
+          || (executing_callback && ! handlevisibility.is ("off")));
 }
 
 graphics_toolkit
@@ -6946,8 +6946,8 @@
 {
   Matrix m = extent.get ().matrix_value ();
 
-  graphics_handle parent = get_parent ();
-  graphics_object parent_obj = gh_manager::get_object (parent);
+  graphics_object parent_obj =
+    gh_manager::get_object (get_parent ());
   Matrix parent_bbox = parent_obj.get_properties ().get_boundingbox (true),
          parent_size = parent_bbox.extract_n (0, 2, 1, 2);
 
@@ -6993,8 +6993,7 @@
 {
   Matrix pos = get_position ().matrix_value ();
 
-  graphics_handle parent = get_parent ();
-  graphics_object parent_obj = gh_manager::get_object (parent);
+  graphics_object parent_obj = gh_manager::get_object (get_parent ());
   Matrix parent_bbox = parent_obj.get_properties ().get_boundingbox (true),
          parent_size = parent_bbox.extract_n (0, 2, 1, 2);
   
@@ -7165,8 +7164,7 @@
 {
   Matrix pos = get_position ().matrix_value ();
 
-  graphics_handle parent = get_parent ();
-  graphics_object parent_obj = gh_manager::get_object (parent);
+  graphics_object parent_obj = gh_manager::get_object (get_parent ());
   Matrix parent_bbox = parent_obj.get_properties ().get_boundingbox (true),
          parent_size = parent_bbox.extract_n (0, 2, 1, 2);
   
@@ -7728,7 +7726,7 @@
 void 
 gh_manager::do_enable_event_processing (bool enable)
 {
-  gh_manager::auto_lock lock;
+  gh_manager::auto_lock guard;
 
   if (enable)
     {
diff -r 2cd903770748 -r deb69dab4514 src/ov.h
--- a/src/ov.h	Thu Oct 20 14:08:25 2011 +0100
+++ b/src/ov.h	Thu Oct 20 15:19:38 2011 +0100
@@ -1373,7 +1373,7 @@
 
 #define DEF_DUMMY_VALUE_EXTRACTOR(VALUE,DEFVAL) \
 template<> \
-inline VALUE octave_value_extract<VALUE> (const octave_value& v) \
+inline VALUE octave_value_extract<VALUE> (const octave_value&) \
   { assert (false); return DEFVAL; }
 
 DEF_DUMMY_VALUE_EXTRACTOR (char, 0)