compilation warning fixed

[wxWidgets.git] / samples / opengl / cube / cube.cpp

diff --git a/samples/opengl/cube/cube.cpp b/samples/opengl/cube/cube.cpp
index 2f145970d23d98a0dbb4396e1b5d236f02c6aa95..a92f940d292dc0edac4f60f7f76d6a2d1b24042c 100644 (file)
--- a/samples/opengl/cube/cube.cpp
+++ b/samples/opengl/cube/cube.cpp
@@ -6,7 +6,7 @@
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart

-// Licence:    wxWindows licence
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__


@@ -27,11 +27,20 @@
 
 #include "wx/log.h"
 
 
 #include "wx/log.h"
 

+#if !wxUSE_GLCANVAS
+#error Please set wxUSE_GLCANVAS to 1 in setup.h.
+#endif
+

 #include "cube.h"
 
 #include "cube.h"
 

-#ifndef __WXMSW__     // for wxStopWatch, see remark below 
-#include <sys/time.h>
-#include <sys/unistd.h>
+#ifndef __WXMSW__     // for wxStopWatch, see remark below
+  #if defined(__WXMAC__) && !defined(__DARWIN__)
+    #include <utime.h>
+    #include <unistd.h>
+  #else
+    #include <sys/time.h>
+    #include <sys/unistd.h>
+  #endif

 #else
 #include <sys/timeb.h>
 #endif
 #else
 #include <sys/timeb.h>
 #endif


@@ -82,19 +91,19 @@ class ScanCodeDialog : public wxDialog
 public:
   ScanCodeDialog( wxWindow* parent, wxWindowID id, const int code,
                   const wxString &descr, const wxString& title );
 public:
   ScanCodeDialog( wxWindow* parent, wxWindowID id, const int code,
                   const wxString &descr, const wxString& title );

-  int GetValue(); 
+  int GetValue();

 private:
   ScanCodeCtrl       *m_ScanCode;
   wxTextCtrl         *m_Description;
 };
 
 private:
   ScanCodeCtrl       *m_ScanCode;
   wxTextCtrl         *m_Description;
 };
 

-ScanCodeDialog::ScanCodeDialog( wxWindow* parent, wxWindowID id, 
+ScanCodeDialog::ScanCodeDialog( wxWindow* parent, wxWindowID id,

                const int code, const wxString &descr, const wxString& title )
           : wxDialog( parent, id, title, wxPoint(-1, -1), wxSize(96*2,76*2) )
 {
                const int code, const wxString &descr, const wxString& title )
           : wxDialog( parent, id, title, wxPoint(-1, -1), wxSize(96*2,76*2) )
 {

-  new wxStaticText( this, -1, "Scancode", wxPoint(4*2,3*2), 
+  new wxStaticText( this, -1, "Scancode", wxPoint(4*2,3*2),

                     wxSize(31*2,12*2) );
                     wxSize(31*2,12*2) );

-  m_ScanCode = new ScanCodeCtrl( this, -1, code, wxPoint(37*2,6*2), 
+  m_ScanCode = new ScanCodeCtrl( this, -1, code, wxPoint(37*2,6*2),

                                  wxSize(53*2,14*2) );
 
   new wxStaticText( this, -1, "Description", wxPoint(4*2,24*2),
                                  wxSize(53*2,14*2) );
 
   new wxStaticText( this, -1, "Description", wxPoint(4*2,24*2),


@@ -117,8 +126,8 @@ int ScanCodeDialog::GetValue()
 
 /*----------------------------------------------------------------------
   Utility function to get the elapsed time (in msec) since a given point
 
 /*----------------------------------------------------------------------
   Utility function to get the elapsed time (in msec) since a given point

-  in time (in sec)  (because current version of wxGetElapsedTime doesn´t 
-  works right with glibc-2.1 and linux, at least for me) 
+  in time (in sec)  (because current version of wxGetElapsedTime doesn´t
+  works right with glibc-2.1 and linux, at least for me)

 -----------------------------------------------------------------------*/
 unsigned long wxStopWatch( unsigned long *sec_base )
 {
 -----------------------------------------------------------------------*/
 unsigned long wxStopWatch( unsigned long *sec_base )
 {


@@ -142,7 +151,7 @@ unsigned long wxStopWatch( unsigned long *sec_base )
 
   if( *sec_base == 0 )
     *sec_base = secs;
 
   if( *sec_base == 0 )
     *sec_base = secs;

-    
+

   return( (secs-*sec_base)*1000 + msec );
 }
 
   return( (secs-*sec_base)*1000 + msec );
 }
 


@@ -174,7 +183,7 @@ TestGLCanvas::TestGLCanvas(wxWindow *parent, wxWindowID id,
     m_rright = WXK_RIGHT;
 }
 
     m_rright = WXK_RIGHT;
 }
 

-TestGLCanvas::TestGLCanvas(wxWindow *parent, const TestGLCanvas &other, 
+TestGLCanvas::TestGLCanvas(wxWindow *parent, const TestGLCanvas &other,

     wxWindowID id, const wxPoint& pos, const wxSize& size, long style,
     const wxString& name ) :
       wxGLCanvas(parent, other.GetContext(), id, pos, size, style, name  )
     wxWindowID id, const wxPoint& pos, const wxSize& size, long style,
     const wxString& name ) :
       wxGLCanvas(parent, other.GetContext(), id, pos, size, style, name  )


@@ -204,14 +213,14 @@ void TestGLCanvas::Render()
         InitGL();
         m_init = TRUE;
     }
         InitGL();
         m_init = TRUE;
     }

-    
+

     /* clear color and depth buffers */
     glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
 
   if( m_gllist == 0 )
   {
     m_gllist = glGenLists( 1 );
     /* clear color and depth buffers */
     glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
 
   if( m_gllist == 0 )
   {
     m_gllist = glGenLists( 1 );

-    glNewList( m_gllist, GL_COMPILE_AND_EXECUTE );        
+    glNewList( m_gllist, GL_COMPILE_AND_EXECUTE );

     /* draw six faces of a cube */
     glBegin(GL_QUADS);
     glNormal3f( 0.0F, 0.0F, 1.0F);
     /* draw six faces of a cube */
     glBegin(GL_QUADS);
     glNormal3f( 0.0F, 0.0F, 1.0F);


@@ -242,7 +251,7 @@ void TestGLCanvas::Render()
     glEndList();
   }
   else
     glEndList();
   }
   else

-    glCallList( m_gllist );    
+    glCallList( m_gllist );

 
   glFlush();
   SwapBuffers();
 
   glFlush();
   SwapBuffers();


@@ -280,7 +289,7 @@ void TestGLCanvas::OnEraseBackground(wxEraseEvent& event)
 void TestGLCanvas::InitGL()
 {
     SetCurrent();
 void TestGLCanvas::InitGL()
 {
     SetCurrent();

-    
+

     /* set viewing projection */
     glMatrixMode(GL_PROJECTION);
     glFrustum(-0.5F, 0.5F, -0.5F, 0.5F, 1.0F, 3.0F);
     /* set viewing projection */
     glMatrixMode(GL_PROJECTION);
     glFrustum(-0.5F, 0.5F, -0.5F, 0.5F, 1.0F, 3.0F);


@@ -322,7 +331,7 @@ GLfloat TestGLCanvas::CalcRotateAngle( unsigned long lasttime,
     t = ((GLfloat)(acceltime - lasttime)) / 1000.0f;
     s1 = CalcRotateSpeed( lasttime );
     s2 = CalcRotateSpeed( acceltime );
     t = ((GLfloat)(acceltime - lasttime)) / 1000.0f;
     s1 = CalcRotateSpeed( lasttime );
     s2 = CalcRotateSpeed( acceltime );

-    
+

     return( t * (s1 + s2) * 135.0f );
 }
 
     return( t * (s1 + s2) * 135.0f );
 }
 


@@ -331,9 +340,9 @@ void TestGLCanvas::Action( long code, unsigned long lasttime,
 {
     GLfloat angle = CalcRotateAngle( lasttime, acceltime );
 
 {
     GLfloat angle = CalcRotateAngle( lasttime, acceltime );
 

-    if (code == m_rleft) 
+    if (code == m_rleft)

         Rotate( angle );
         Rotate( angle );

-    else if (code == m_rright) 
+    else if (code == m_rright)

             Rotate( -angle );
 }
 
             Rotate( -angle );
 }
 


@@ -355,8 +364,11 @@ void TestGLCanvas::OnKeyDown( wxKeyEvent& event )
     }
 
     unsigned long currTime = event.m_timeStamp - m_xsynct;
     }
 
     unsigned long currTime = event.m_timeStamp - m_xsynct;

-  
-    if (evkey != m_Key)
+
+    // we have to test for m_Key != 0 because otherwise the test would be
+    // always true because it is set to 0 in OnKeyUp() below - I don't know
+    // why is it like this, just fixing blindly (VZ)
+    if (evkey != m_Key && m_Key != 0)

     {
         m_Key = evkey;
         m_LastRedraw = m_StartTime = m_LastTime = currTime;
     {
         m_Key = evkey;
         m_LastRedraw = m_StartTime = m_LastTime = currTime;


@@ -365,11 +377,11 @@ void TestGLCanvas::OnKeyDown( wxKeyEvent& event )
     if (currTime >= m_LastRedraw)      // Redraw:
     {
         Action( m_Key, m_LastTime-m_StartTime, currTime-m_StartTime );
     if (currTime >= m_LastRedraw)      // Redraw:
     {
         Action( m_Key, m_LastTime-m_StartTime, currTime-m_StartTime );

-                  
+

         m_LastRedraw = wxStopWatch(&m_secbase) - m_gsynct;
         m_LastTime = currTime;
     }
         m_LastRedraw = wxStopWatch(&m_secbase) - m_gsynct;
         m_LastTime = currTime;
     }

-    
+

     event.Skip();
 }
 
     event.Skip();
 }
 


@@ -379,7 +391,7 @@ void TestGLCanvas::OnKeyUp( wxKeyEvent& event )
     m_StartTime = 0;
     m_LastTime = 0;
     m_LastRedraw = 0;
     m_StartTime = 0;
     m_LastTime = 0;
     m_LastRedraw = 0;

-    
+

     event.Skip();
 }
 
     event.Skip();
 }
 


@@ -406,7 +418,7 @@ END_EVENT_TABLE()
 
 // My frame constructor
 MyFrame::MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos,
 
 // My frame constructor
 MyFrame::MyFrame(wxFrame *frame, const wxString& title, const wxPoint& pos,

-                 const wxSize& size, long style) 
+                 const wxSize& size, long style)

          : wxFrame(frame, -1, title, pos, size, style)
 {
     m_canvas = NULL;
          : wxFrame(frame, -1, title, pos, size, style)
 {
     m_canvas = NULL;


@@ -423,7 +435,7 @@ void MyFrame::OnNewWindow()
   MyFrame *frame = new MyFrame(NULL, "Cube OpenGL Demo Clone",
                                wxPoint(50, 50), wxSize(400, 300));
   // Give it an icon
   MyFrame *frame = new MyFrame(NULL, "Cube OpenGL Demo Clone",
                                wxPoint(50, 50), wxSize(400, 300));
   // Give it an icon

-#ifdef wx_msw
+#ifdef __WXMSW__

   frame->SetIcon(wxIcon("mondrian"));
 #endif
 
   frame->SetIcon(wxIcon("mondrian"));
 #endif
 


@@ -442,16 +454,16 @@ void MyFrame::OnNewWindow()
 
   frame->SetMenuBar(menuBar);
 
 
   frame->SetMenuBar(menuBar);
 

-  frame->m_canvas = new TestGLCanvas( frame, *m_canvas, -1, 
+  frame->m_canvas = new TestGLCanvas( frame, *m_canvas, -1,

                wxPoint(0, 0), wxSize(200, 200) );
                wxPoint(0, 0), wxSize(200, 200) );

-  
+

   // Show the frame
   frame->Show(TRUE);
 }
 
 void MyFrame::OnDefRotateLeftKey()
 {
   // Show the frame
   frame->Show(TRUE);
 }
 
 void MyFrame::OnDefRotateLeftKey()
 {

-  ScanCodeDialog dial( this, -1, m_canvas->m_rleft, 
+  ScanCodeDialog dial( this, -1, m_canvas->m_rleft,

                        wxString("Left"), "Define key" );
   int result = dial.ShowModal();
   if( result == wxID_OK )
                        wxString("Left"), "Define key" );
   int result = dial.ShowModal();
   if( result == wxID_OK )


@@ -459,7 +471,7 @@ void MyFrame::OnDefRotateLeftKey()
 }
 void MyFrame::OnDefRotateRightKey()
 {
 }
 void MyFrame::OnDefRotateRightKey()
 {

-  ScanCodeDialog dial( this, -1, m_canvas->m_rright, 
+  ScanCodeDialog dial( this, -1, m_canvas->m_rright,

                        wxString("Right"), "Define key" );
   int result = dial.ShowModal();
   if( result == wxID_OK )
                        wxString("Right"), "Define key" );
   int result = dial.ShowModal();
   if( result == wxID_OK )


@@ -503,6 +515,6 @@ bool MyApp::OnInit(void)
 
   // Show the frame
   frame->Show(TRUE);
 
   // Show the frame
   frame->Show(TRUE);

-  
+

   return TRUE;
 }
   return TRUE;
 }