compilation fix

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

diff --git a/samples/opengl/isosurf/isosurf.cpp b/samples/opengl/isosurf/isosurf.cpp
index 3a5cc72b01b26cb064e7910abd890902f70c3a84..8e5aec635773e2eadc192cdafb9f57aeda88dd39 100644 (file)
--- a/samples/opengl/isosurf/isosurf.cpp
+++ b/samples/opengl/isosurf/isosurf.cpp
@@ -6,14 +6,9 @@
 // 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__
-#pragma implementation
-#pragma interface
-#endif
-

 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 


@@ -26,20 +21,37 @@
 #endif
 
 #if !wxUSE_GLCANVAS
 #endif
 
 #if !wxUSE_GLCANVAS

-#error Please set wxUSE_GLCANVAS to 1 in setup.h.
+    #error "OpenGL required: set wxUSE_GLCANVAS to 1 and rebuild the library"

 #endif
 
 #include "wx/timer.h"
 #include "wx/glcanvas.h"
 #endif
 
 #include "wx/timer.h"
 #include "wx/glcanvas.h"

+#include "wx/math.h"
+
+#if defined(__WXMAC__) || defined(__WXCOCOA__)
+#   ifdef __DARWIN__
+#       include <OpenGL/gl.h>
+#       include <OpenGL/glu.h>
+#   else
+#       include <gl.h>
+#       include <glu.h>
+#   endif
+#else
+#   include <GL/gl.h>
+#   include <GL/glu.h>
+#endif

 
 

-#include <GL/gl.h>
-#include <GL/glu.h>
+// disabled because this has apparently changed in OpenGL 1.2, so doesn't link
+// correctly if this is on...
+#ifdef GL_EXT_vertex_array
+#undef GL_EXT_vertex_array
+#endif

 
 #include "isosurf.h"
 
 
 #include "isosurf.h"
 

-// The following part is taken largely unchanged from the original C Version
+#include "../../sample.xpm"

 
 

-#include <math.h>
+// The following part is taken largely unchanged from the original C Version

 
 GLboolean speed_test = GL_FALSE;
 GLboolean use_vertex_arrays = GL_FALSE;
 
 GLboolean speed_test = GL_FALSE;
 GLboolean use_vertex_arrays = GL_FALSE;


@@ -60,60 +72,63 @@ static GLfloat xrot;
 static GLfloat yrot;
 
 
 static GLfloat yrot;
 
 

-static void read_surface( char *filename )
+static void read_surface( const wxChar *filename )

 {
 {

-   FILE *f;
-
-   f = fopen(filename,"r");
-   if (!f) {
-      wxString msg("Couldn't read ");
-      msg += filename;
-      wxMessageBox(msg);
-      return;
-   }
-
-   numverts = 0;
-   while (!feof(f) && numverts<MAXVERTS) {
-      fscanf( f, "%f %f %f  %f %f %f",
-             &verts[numverts][0], &verts[numverts][1], &verts[numverts][2],
-             &norms[numverts][0], &norms[numverts][1], &norms[numverts][2] );
-      numverts++;
-   }
-   numverts--;
-
-   printf("%d vertices, %d triangles\n", numverts, numverts-2);
-   fclose(f);
+    FILE *f = wxFopen(filename,_T("r"));
+    if (!f)
+    {
+        wxString msg = _T("Couldn't read ");
+        msg += filename;
+        wxMessageBox(msg);
+        return;
+    }
+
+    numverts = 0;
+    while (!feof(f) && numverts<MAXVERTS)
+    {
+        fscanf( f, "%f %f %f  %f %f %f",
+            &verts[numverts][0], &verts[numverts][1], &verts[numverts][2],
+            &norms[numverts][0], &norms[numverts][1], &norms[numverts][2] );
+        numverts++;
+    }
+
+    numverts--;
+
+    wxPrintf(_T("%d vertices, %d triangles\n"), numverts, numverts-2);
+
+    fclose(f);

 }
 
 
 }
 
 

-static void draw_surface( void )
+static void draw_surface()

 {
 {

-   GLint i;
+    GLint i;

 
 #ifdef GL_EXT_vertex_array
 
 #ifdef GL_EXT_vertex_array

-   if (use_vertex_arrays) {
-      glDrawArraysEXT( GL_TRIANGLE_STRIP, 0, numverts );
-   }
-   else {
-#endif
-      glBegin( GL_TRIANGLE_STRIP );
-      for (i=0;i<numverts;i++) {
-         glNormal3fv( norms[i] );
-         glVertex3fv( verts[i] );
-      }
-      glEnd();
-#ifdef GL_EXT_vertex_array
-   }
+    if (use_vertex_arrays)
+    {
+        glDrawArraysEXT( GL_TRIANGLE_STRIP, 0, numverts );
+    }
+    else

 #endif
 #endif

+    {
+        glBegin( GL_TRIANGLE_STRIP );
+        for (i=0;i<numverts;i++)
+        {
+            glNormal3fv( norms[i] );
+            glVertex3fv( verts[i] );
+        }
+        glEnd();
+    }

 }
 
 
 }
 
 

-static void draw1(void)
+static void draw1()

 {
     glClear( GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT );
     glPushMatrix();
 {
     glClear( GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT );
     glPushMatrix();

-    glRotatef( yrot, 0.0, 1.0, 0.0 );
-    glRotatef( xrot, 1.0, 0.0, 0.0 );
+    glRotatef( yrot, 0.0f, 1.0f, 0.0f );
+    glRotatef( xrot, 1.0f, 0.0f, 0.0f );

 
     draw_surface();
 
 
     draw_surface();
 


@@ -123,33 +138,33 @@ static void draw1(void)
 }
 
 
 }
 
 

-static void InitMaterials(void)
+static void InitMaterials()

 {
 {

-    static float ambient[] = {0.1, 0.1, 0.1, 1.0};
-    static float diffuse[] = {0.5, 1.0, 1.0, 1.0};
-    static float position0[] = {0.0, 0.0, 20.0, 0.0};
-    static float position1[] = {0.0, 0.0, -20.0, 0.0};
-    static float front_mat_shininess[] = {60.0};
-    static float front_mat_specular[] = {0.2, 0.2, 0.2, 1.0};
-    static float front_mat_diffuse[] = {0.5, 0.28, 0.38, 1.0};
+    static const GLfloat ambient[4] = {0.1f, 0.1f, 0.1f, 1.0f};
+    static const GLfloat diffuse[4] = {0.5f, 1.0f, 1.0f, 1.0f};
+    static const GLfloat position0[4] = {0.0f, 0.0f, 20.0f, 0.0f};
+    static const GLfloat position1[4] = {0.0f, 0.0f, -20.0f, 0.0f};
+    static const GLfloat front_mat_shininess[1] = {60.0f};
+    static const GLfloat front_mat_specular[4] = {0.2f, 0.2f, 0.2f, 1.0f};
+    static const GLfloat front_mat_diffuse[4] = {0.5f, 0.28f, 0.38f, 1.0f};

     /*
     /*

-    static float back_mat_shininess[] = {60.0};
-    static float back_mat_specular[] = {0.5, 0.5, 0.2, 1.0};
-    static float back_mat_diffuse[] = {1.0, 1.0, 0.2, 1.0};
+    static const GLfloat back_mat_shininess[1] = {60.0f};
+    static const GLfloat back_mat_specular[4] = {0.5f, 0.5f, 0.2f, 1.0f};
+    static const GLfloat back_mat_diffuse[4] = {1.0f, 1.0f, 0.2f, 1.0f};

     */
     */

-    static float lmodel_ambient[] = {1.0, 1.0, 1.0, 1.0};
-    static float lmodel_twoside[] = {GL_FALSE};
+    static const GLfloat lmodel_ambient[4] = {1.0f, 1.0f, 1.0f, 1.0f};
+    static const GLfloat lmodel_twoside[1] = {GL_FALSE};

 
     glLightfv(GL_LIGHT0, GL_AMBIENT, ambient);
     glLightfv(GL_LIGHT0, GL_DIFFUSE, diffuse);
     glLightfv(GL_LIGHT0, GL_POSITION, position0);
     glEnable(GL_LIGHT0);
 
     glLightfv(GL_LIGHT0, GL_AMBIENT, ambient);
     glLightfv(GL_LIGHT0, GL_DIFFUSE, diffuse);
     glLightfv(GL_LIGHT0, GL_POSITION, position0);
     glEnable(GL_LIGHT0);

-    
+

     glLightfv(GL_LIGHT1, GL_AMBIENT, ambient);
     glLightfv(GL_LIGHT1, GL_DIFFUSE, diffuse);
     glLightfv(GL_LIGHT1, GL_POSITION, position1);
     glEnable(GL_LIGHT1);
     glLightfv(GL_LIGHT1, GL_AMBIENT, ambient);
     glLightfv(GL_LIGHT1, GL_DIFFUSE, diffuse);
     glLightfv(GL_LIGHT1, GL_POSITION, position1);
     glEnable(GL_LIGHT1);

-    
+

     glLightModelfv(GL_LIGHT_MODEL_AMBIENT, lmodel_ambient);
     glLightModelfv(GL_LIGHT_MODEL_TWO_SIDE, lmodel_twoside);
     glEnable(GL_LIGHTING);
     glLightModelfv(GL_LIGHT_MODEL_AMBIENT, lmodel_ambient);
     glLightModelfv(GL_LIGHT_MODEL_TWO_SIDE, lmodel_twoside);
     glEnable(GL_LIGHTING);


@@ -162,122 +177,129 @@ static void InitMaterials(void)
 
 static void Init(void)
 {
 
 static void Init(void)
 {

-   glClearColor(0.0, 0.0, 0.0, 0.0);
+    glClearColor(0.0f, 0.0f, 0.0f, 0.0f);

 
 

-   glShadeModel(GL_SMOOTH);
-   glEnable(GL_DEPTH_TEST);
+    glShadeModel(GL_SMOOTH);
+    glEnable(GL_DEPTH_TEST);

 
 

-   InitMaterials();
+    InitMaterials();

 
 

-   glMatrixMode(GL_PROJECTION);
-   glLoadIdentity();
-   glFrustum( -1.0, 1.0, -1.0, 1.0, 5, 25 );
+    glMatrixMode(GL_PROJECTION);
+    glLoadIdentity();
+    glFrustum( -1.0, 1.0, -1.0, 1.0, 5.0, 25.0 );

 
 

-   glMatrixMode(GL_MODELVIEW);
-   glLoadIdentity();
-   glTranslatef( 0.0, 0.0, -6.0 );
+    glMatrixMode(GL_MODELVIEW);
+    glLoadIdentity();
+    glTranslatef( 0.0, 0.0, -6.0 );

 
 #ifdef GL_EXT_vertex_array
 
 #ifdef GL_EXT_vertex_array

-   if (use_vertex_arrays) {
-      glVertexPointerEXT( 3, GL_FLOAT, 0, numverts, verts );
-      glNormalPointerEXT( GL_FLOAT, 0, numverts, norms );
-      glEnable( GL_VERTEX_ARRAY_EXT );
-      glEnable( GL_NORMAL_ARRAY_EXT );
-   }
+    if (use_vertex_arrays)
+    {
+        glVertexPointerEXT( 3, GL_FLOAT, 0, numverts, verts );
+        glNormalPointerEXT( GL_FLOAT, 0, numverts, norms );
+        glEnable( GL_VERTEX_ARRAY_EXT );
+        glEnable( GL_NORMAL_ARRAY_EXT );
+    }

 #endif
 }
 
 #endif
 }
 

-
-static void Reshape(int width, int height)
+static GLenum Args(int argc, wxChar **argv)

 {
 {

-  glViewport(0, 0, (GLint)width, (GLint)height);
-}
-
+    GLint i;
+
+    for (i = 1; i < argc; i++)
+    {
+        if (wxStrcmp(argv[i], _T("-sb")) == 0)
+        {
+            doubleBuffer = GL_FALSE;
+        }
+        else if (wxStrcmp(argv[i], _T("-db")) == 0)
+        {
+            doubleBuffer = GL_TRUE;
+        }
+        else if (wxStrcmp(argv[i], _T("-speed")) == 0)
+        {
+            speed_test = GL_TRUE;
+            doubleBuffer = GL_TRUE;
+        }
+        else if (wxStrcmp(argv[i], _T("-va")) == 0)
+        {
+            use_vertex_arrays = GL_TRUE;
+        }
+        else
+        {
+            wxString msg = _T("Bad option: ");
+            msg += argv[i];
+            wxMessageBox(msg);
+            return GL_FALSE;
+        }
+    }

 
 

-static GLenum Args(int argc, char **argv)
-{
-   GLint i;
-
-   for (i = 1; i < argc; i++) {
-      if (strcmp(argv[i], "-sb") == 0) {
-         doubleBuffer = GL_FALSE;
-      }
-      else if (strcmp(argv[i], "-db") == 0) {
-         doubleBuffer = GL_TRUE;
-      }
-      else if (strcmp(argv[i], "-speed") == 0) {
-         speed_test = GL_TRUE;
-         doubleBuffer = GL_TRUE;
-      }
-      else if (strcmp(argv[i], "-va") == 0) {
-         use_vertex_arrays = GL_TRUE;
-      }
-      else {
-         wxString msg("Bad option: ");
-         msg += argv[i];
-         wxMessageBox(msg);
-         return GL_FALSE;
-      }
-   }
-
-   return GL_TRUE;
+    return GL_TRUE;

 }
 
 }
 

-// The following part was written for wxWindows 1.66
+// The following part was written for wxWidgets 1.66

 MyFrame *frame = NULL;
 
 IMPLEMENT_APP(MyApp)
 
 // `Main program' equivalent, creating windows and returning main app frame
 MyFrame *frame = NULL;
 
 IMPLEMENT_APP(MyApp)
 
 // `Main program' equivalent, creating windows and returning main app frame

-bool MyApp::OnInit(void)
+bool MyApp::OnInit()

 {
 {

-  Args(argc, argv);
+    Args(argc, argv);

 
 

-  // Create the main frame window
-  frame = new MyFrame(NULL, "Isosurf GL Sample", wxPoint(50, 50), wxSize(200, 200));
+    // Create the main frame window
+    frame = new MyFrame(NULL, wxT("wxWidgets OpenGL Isosurf Sample"),
+        wxDefaultPosition, wxDefaultSize);

 
 

-  // Give it an icon
-  frame->SetIcon(wxIcon("mondrian"));
+    // Give it an icon
+    frame->SetIcon(wxIcon(_T("mondrian")));

 
 

-  // Make a menubar
-  wxMenu *fileMenu = new wxMenu;
+    // Make a menubar
+    wxMenu *fileMenu = new wxMenu;

 
 

-  fileMenu->Append(wxID_EXIT, "E&xit");
-  wxMenuBar *menuBar = new wxMenuBar;
-  menuBar->Append(fileMenu, "&File");
-  frame->SetMenuBar(menuBar);
+    fileMenu->Append(wxID_EXIT, _T("E&xit"));
+    wxMenuBar *menuBar = new wxMenuBar;
+    menuBar->Append(fileMenu, _T("&File"));
+    frame->SetMenuBar(menuBar);

 
   // Make a TestGLCanvas
 
   // JACS
 #ifdef __WXMSW__
 
   // Make a TestGLCanvas
 
   // JACS
 #ifdef __WXMSW__

-  int *gl_attrib = NULL;
+    int *gl_attrib = NULL;

 #else
 #else

-  int gl_attrib[20] = { GLX_RGBA, GLX_RED_SIZE, 1, GLX_GREEN_SIZE, 1,
-                       GLX_BLUE_SIZE, 1, GLX_DEPTH_SIZE, 1,
-                       GLX_DOUBLEBUFFER, None };
+    int gl_attrib[20] = { WX_GL_RGBA, WX_GL_MIN_RED, 1, WX_GL_MIN_GREEN, 1,
+        WX_GL_MIN_BLUE, 1, WX_GL_DEPTH_SIZE, 1,
+        WX_GL_DOUBLEBUFFER,
+#  ifdef __WXMAC__
+        GL_NONE };
+#  else
+        None };
+#  endif

 #endif
 
 #endif
 

-  if(!doubleBuffer)
-   {
-      printf("don't have double buffer, disabling\n");
+    if(!doubleBuffer)
+    {
+        printf("don't have double buffer, disabling\n");

 #ifdef __WXGTK__
 #ifdef __WXGTK__

-      gl_attrib[9] = None;
+        gl_attrib[9] = None;

 #endif
 #endif

-      doubleBuffer = GL_FALSE;
-  }
-  frame->m_canvas = new TestGLCanvas(frame, -1, wxPoint(0, 0), wxSize(200, 200), 0, "TestGLCanvas",
-                                  gl_attrib);
+        doubleBuffer = GL_FALSE;
+    }
+
+    frame->m_canvas = new TestGLCanvas(frame, wxID_ANY, wxDefaultPosition,
+        wxDefaultSize, 0, _T("TestGLCanvas"), gl_attrib );

 
   // Show the frame
 
   // Show the frame

-  frame->Show(TRUE);
+    frame->Show(true);

 
 

-  frame->m_canvas->SetCurrent();
-  read_surface( "isosurf.dat" );
+    frame->m_canvas->SetCurrent();
+    read_surface( _T("isosurf.dat") );

 
 

-  Init();
+    Init();

 
 

-  return TRUE;
+    return true;

 }
 
 BEGIN_EVENT_TABLE(MyFrame, wxFrame)
 }
 
 BEGIN_EVENT_TABLE(MyFrame, wxFrame)


@@ -286,16 +308,23 @@ 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):
-  wxFrame(frame, -1, title, pos, size, style)
+    const wxSize& size, long style)
+    : wxFrame(frame, wxID_ANY, title, pos, size, style)

 {
     m_canvas = NULL;
 {
     m_canvas = NULL;

+    SetIcon(wxIcon(sample_xpm));
+}
+
+MyFrame::~MyFrame()
+{
+    delete m_canvas;

 }
 
 // Intercept menu commands
 }
 
 // Intercept menu commands

-void MyFrame::OnExit(wxCommandEvent& event)
+void MyFrame::OnExit( wxCommandEvent& WXUNUSED(event) )

 {
 {

-    Destroy();
+    // true is to force the frame to close
+    Close(true);

 }
 
 /*
 }
 
 /*


@@ -311,24 +340,23 @@ BEGIN_EVENT_TABLE(TestGLCanvas, wxGLCanvas)
 END_EVENT_TABLE()
 
 TestGLCanvas::TestGLCanvas(wxWindow *parent, wxWindowID id,
 END_EVENT_TABLE()
 
 TestGLCanvas::TestGLCanvas(wxWindow *parent, wxWindowID id,

-    const wxPoint& pos, const wxSize& size, long style, const wxString& name, int* gl_attrib):
-  wxGLCanvas(parent, id, pos, size, style, name, gl_attrib)
+    const wxPoint& pos, const wxSize& size, long style,
+    const wxString& name, int* gl_attrib)
+    : wxGLCanvas(parent, id, pos, size, style|wxFULL_REPAINT_ON_RESIZE, name, gl_attrib)

 {
 {

-   parent->Show(TRUE);
-   SetCurrent();
-   /* Make sure server supports the vertex array extension */
-   char* extensions = (char *) glGetString( GL_EXTENSIONS );
-   if (!extensions || !strstr( extensions, "GL_EXT_vertex_array" )) {
-      use_vertex_arrays = GL_FALSE;
-   }
-}
-
+    parent->Show(true);
+    SetCurrent();

 
 

-TestGLCanvas::~TestGLCanvas(void)
-{
+    /* Make sure server supports the vertex array extension */
+    char* extensions = (char *) glGetString( GL_EXTENSIONS );
+    if (!extensions || !strstr( extensions, "GL_EXT_vertex_array" ))
+    {
+        use_vertex_arrays = GL_FALSE;
+    }

 }
 
 }
 

-void TestGLCanvas::OnPaint( wxPaintEvent& event )
+
+void TestGLCanvas::OnPaint( wxPaintEvent& WXUNUSED(event) )

 {
     // This is a dummy, to avoid an endless succession of paint messages.
     // OnPaint handlers must always create a wxPaintDC.
 {
     // This is a dummy, to avoid an endless succession of paint messages.
     // OnPaint handlers must always create a wxPaintDC.


@@ -338,63 +366,83 @@ void TestGLCanvas::OnPaint( wxPaintEvent& event )
     if (!GetContext()) return;
 #endif
 
     if (!GetContext()) return;
 #endif
 

+    SetCurrent();
+

     draw1();
     SwapBuffers();
 }
 
 void TestGLCanvas::OnSize(wxSizeEvent& event)
 {
     draw1();
     SwapBuffers();
 }
 
 void TestGLCanvas::OnSize(wxSizeEvent& event)
 {

+    // this is also necessary to update the context on some platforms
+    wxGLCanvas::OnSize(event);
+
+    // set GL viewport (not called by wxGLCanvas::OnSize on all platforms...)
+    int w, h;
+    GetClientSize(&w, &h);

 #ifndef __WXMOTIF__
 #ifndef __WXMOTIF__

-    if (!GetContext()) return;
+    if (GetContext())

 #endif
 #endif

-
-    SetCurrent();
-    int width, height;
-    GetClientSize(& width, & height);
-    Reshape(width, height);
+    {
+        SetCurrent();
+        glViewport(0, 0, (GLint) w, (GLint) h);
+    }

 }
 
 void TestGLCanvas::OnChar(wxKeyEvent& event)
 {
 }
 
 void TestGLCanvas::OnChar(wxKeyEvent& event)
 {

-    switch(event.KeyCode()) {
+    switch( event.GetKeyCode() )
+    {

     case WXK_ESCAPE:
     case WXK_ESCAPE:

-       exit(0);
+        wxTheApp->ExitMainLoop();
+        return;
+

     case WXK_LEFT:
     case WXK_LEFT:

-       yrot -= 15.0;
-       break;
+        yrot -= 15.0;
+        break;
+

     case WXK_RIGHT:
     case WXK_RIGHT:

-       yrot += 15.0;
-       break;
+        yrot += 15.0;
+        break;
+

     case WXK_UP:
     case WXK_UP:

-       xrot += 15.0;
-       break;
+        xrot += 15.0;
+        break;
+

     case WXK_DOWN:
     case WXK_DOWN:

-       xrot -= 15.0;
-       break;
+        xrot -= 15.0;
+        break;
+

     case 's': case 'S':
     case 's': case 'S':

-       smooth = !smooth;
-       if (smooth) {
-           glShadeModel(GL_SMOOTH);
-       } else {
-           glShadeModel(GL_FLAT);
-       }
-       break;
+        smooth = !smooth;
+        if (smooth)
+        {
+            glShadeModel(GL_SMOOTH);
+        }
+        else
+        {
+            glShadeModel(GL_FLAT);
+        }
+        break;
+

     case 'l': case 'L':
     case 'l': case 'L':

-       lighting = !lighting;
-       if (lighting) {
-           glEnable(GL_LIGHTING);
-       } else {
-           glDisable(GL_LIGHTING);
-       }
-       break;
-     default:
-      {
+        lighting = !lighting;
+        if (lighting)
+        {
+            glEnable(GL_LIGHTING);
+        }
+        else
+        {
+            glDisable(GL_LIGHTING);
+        }
+        break;
+
+    default:

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

-       return;
-      }
+        return;

     }
 
     }
 

-    Refresh(FALSE);
+    Refresh(false);

 }
 
 void TestGLCanvas::OnMouseEvent(wxMouseEvent& event)
 }
 
 void TestGLCanvas::OnMouseEvent(wxMouseEvent& event)


@@ -403,21 +451,27 @@ void TestGLCanvas::OnMouseEvent(wxMouseEvent& event)
     static float last_x, last_y;
 
     //printf("%f %f %d\n", event.GetX(), event.GetY(), (int)event.LeftIsDown());
     static float last_x, last_y;
 
     //printf("%f %f %d\n", event.GetX(), event.GetY(), (int)event.LeftIsDown());

-    if(event.LeftIsDown()) {
-       if(!dragging) {
-           dragging = 1;
-       } else {
-           yrot += (event.GetX() - last_x)*1.0;
-           xrot += (event.GetY() - last_y)*1.0;
-           Refresh(FALSE);
-       }
-       last_x = event.GetX();
-       last_y = event.GetY();
-    } else
-       dragging = 0;
+    if(event.LeftIsDown())
+    {
+        if(!dragging)
+        {
+            dragging = 1;
+        }
+        else
+        {
+            yrot += (event.GetX() - last_x)*1.0;
+            xrot += (event.GetY() - last_y)*1.0;
+            Refresh(false);
+        }
+        last_x = event.GetX();
+        last_y = event.GetY();
+    }
+    else
+        dragging = 0;
+

 }
 
 }
 

-void TestGLCanvas::OnEraseBackground(wxEraseEvent& event)
+void TestGLCanvas::OnEraseBackground( wxEraseEvent& WXUNUSED(event) )

 {
     // Do nothing, to avoid flashing.
 }
 {
     // Do nothing, to avoid flashing.
 }