Commit 77d173b8 authored by Cyril Deguet's avatar Cyril Deguet

* interface.cpp: fixed a stack corruption which caused a segfault in the

  wxwindows plugin (in UpdateAcceleratorTable, the table
  wxAcceleratorEntry entries[6] was filled with _seven_ values ;-)
  I guess it didn't happen with all compilers because of the way
  variables are stored on the stack)
* wxwindows.h: removed a warning
parent 157a5251
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
* interface.cpp : wxWindows plugin for vlc * interface.cpp : wxWindows plugin for vlc
***************************************************************************** *****************************************************************************
* Copyright (C) 2000-2001 VideoLAN * Copyright (C) 2000-2001 VideoLAN
* $Id: interface.cpp,v 1.55 2003/08/14 19:25:56 sigmunau Exp $ * $Id: interface.cpp,v 1.56 2003/08/22 08:06:25 asmax Exp $
* *
* Authors: Gildas Bazin <gbazin@netcourrier.com> * Authors: Gildas Bazin <gbazin@netcourrier.com>
* *
...@@ -363,7 +363,7 @@ void Interface::CreateOurToolBar() ...@@ -363,7 +363,7 @@ void Interface::CreateOurToolBar()
* version because we don't include wx.rc */ * version because we don't include wx.rc */
wxToolBar *toolbar = CreateToolBar( wxToolBar *toolbar = CreateToolBar(
wxTB_HORIZONTAL | wxTB_FLAT | wxTB_DOCKABLE ); wxTB_HORIZONTAL | wxTB_FLAT | wxTB_DOCKABLE );
toolbar->SetToolBitmapSize( wxSize(TOOLBAR_BMP_WIDTH,TOOLBAR_BMP_HEIGHT) ); toolbar->SetToolBitmapSize( wxSize(TOOLBAR_BMP_WIDTH,TOOLBAR_BMP_HEIGHT) );
...@@ -453,7 +453,7 @@ void Interface::CreateOurSlider() ...@@ -453,7 +453,7 @@ void Interface::CreateOurSlider()
void Interface::UpdateAcceleratorTable() void Interface::UpdateAcceleratorTable()
{ {
/* Set some hotkeys */ /* Set some hotkeys */
wxAcceleratorEntry entries[6]; wxAcceleratorEntry entries[7];
int i_key = config_GetInt( p_intf, "quit-key" ); int i_key = config_GetInt( p_intf, "quit-key" );
int i = 0; int i = 0;
entries[i++].Set( ConvertHotkeyModifiers( i_key ), ConvertHotkey( i_key ), entries[i++].Set( ConvertHotkeyModifiers( i_key ), ConvertHotkey( i_key ),
...@@ -477,7 +477,7 @@ void Interface::UpdateAcceleratorTable() ...@@ -477,7 +477,7 @@ void Interface::UpdateAcceleratorTable()
entries[i++].Set( ConvertHotkeyModifiers( i_key ), ConvertHotkey( i_key ), entries[i++].Set( ConvertHotkeyModifiers( i_key ), ConvertHotkey( i_key ),
SlowStream_Event ); SlowStream_Event );
wxAcceleratorTable accel( 6, entries ); wxAcceleratorTable accel( 7, entries );
if( !accel.Ok() ) if( !accel.Ok() )
msg_Err( p_intf, "invalid accelerator table" ); msg_Err( p_intf, "invalid accelerator table" );
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
* wxwindows.h: private wxWindows interface description * wxwindows.h: private wxWindows interface description
***************************************************************************** *****************************************************************************
* Copyright (C) 1999, 2000 VideoLAN * Copyright (C) 1999, 2000 VideoLAN
* $Id: wxwindows.h,v 1.55 2003/08/20 19:17:29 adn Exp $ * $Id: wxwindows.h,v 1.56 2003/08/22 08:06:25 asmax Exp $
* *
* Authors: Gildas Bazin <gbazin@netcourrier.com> * Authors: Gildas Bazin <gbazin@netcourrier.com>
* *
...@@ -762,8 +762,7 @@ static inline int ConvertHotkey( int i_hotkey ) ...@@ -762,8 +762,7 @@ static inline int ConvertHotkey( int i_hotkey )
case KEY_PAGEDOWN: return WXK_NEXT; case KEY_PAGEDOWN: return WXK_NEXT;
case KEY_TAB: return WXK_TAB; case KEY_TAB: return WXK_TAB;
case KEY_BACKSPACE: return WXK_BACK; case KEY_BACKSPACE: return WXK_BACK;
default:
return 0;
} }
} }
return 0;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment