Commit fb9fadde authored by Clément Stenac's avatar Clément Stenac

Skins strings (Refs:#438)

parent f7e4ba30
...@@ -45,7 +45,7 @@ void CmdChangeSkin::execute() ...@@ -45,7 +45,7 @@ void CmdChangeSkin::execute()
if( loader.load( m_file ) ) if( loader.load( m_file ) )
{ {
// Everything went well // Everything went well
msg_Dbg( getIntf(), "New theme successfully loaded (%s)", msg_Info( getIntf(), "new theme successfully loaded (%s)",
m_file.c_str() ); m_file.c_str() );
if( pOldTheme ) if( pOldTheme )
{ {
...@@ -54,14 +54,14 @@ void CmdChangeSkin::execute() ...@@ -54,14 +54,14 @@ void CmdChangeSkin::execute()
} }
else if( pOldTheme ) else if( pOldTheme )
{ {
msg_Err( getIntf(), "A problem occurred when loading the new theme," msg_Warn( getIntf(), "a problem occurred when loading the new theme,"
" restoring the previous one" ); " restoring the previous one" );
getIntf()->p_sys->p_theme = pOldTheme; getIntf()->p_sys->p_theme = pOldTheme;
pOldTheme->getWindowManager().showAll(); pOldTheme->getWindowManager().showAll();
} }
else else
{ {
msg_Err( getIntf(), "Cannot load the theme, aborting" ); msg_Err( getIntf(), "cannot load the theme, aborting" );
// Quit // Quit
CmdQuit cmd( getIntf() ); CmdQuit cmd( getIntf() );
cmd.execute(); cmd.execute();
......
...@@ -119,7 +119,7 @@ class CmdDialogs: public CmdGeneric ...@@ -119,7 +119,7 @@ class CmdDialogs: public CmdGeneric
pDialogs->showStreamingWizard(); pDialogs->showStreamingWizard();
break; break;
default: default:
msg_Warn( getIntf(), "Unknown dialog type" ); msg_Warn( getIntf(), "unknown dialog type" );
break; break;
} }
} }
......
...@@ -37,7 +37,7 @@ const string EvtMouse::getAsString() const ...@@ -37,7 +37,7 @@ const string EvtMouse::getAsString() const
else if( m_button == kRight ) else if( m_button == kRight )
event += ":right"; event += ":right";
else else
msg_Warn( getIntf(), "Unknown button type" ); msg_Warn( getIntf(), "unknown button type" );
// Add the action // Add the action
if( m_action == kDown ) if( m_action == kDown )
...@@ -47,7 +47,7 @@ const string EvtMouse::getAsString() const ...@@ -47,7 +47,7 @@ const string EvtMouse::getAsString() const
else if( m_action == kDblClick ) else if( m_action == kDblClick )
event += ":dblclick"; event += ":dblclick";
else else
msg_Warn( getIntf(), "Unknown action type" ); msg_Warn( getIntf(), "unknown action type" );
// Add the modifier // Add the modifier
addModifier( event ); addModifier( event );
......
...@@ -35,7 +35,7 @@ const string EvtScroll::getAsString() const ...@@ -35,7 +35,7 @@ const string EvtScroll::getAsString() const
else if( m_direction == kDown ) else if( m_direction == kDown )
event += ":down"; event += ":down";
else else
msg_Warn( getIntf(), "Unknown scrolling direction" ); msg_Warn( getIntf(), "unknown scrolling direction" );
// Add the modifier // Add the modifier
addModifier( event ); addModifier( event );
......
...@@ -39,7 +39,7 @@ const string EvtSpecial::getAsString() const ...@@ -39,7 +39,7 @@ const string EvtSpecial::getAsString() const
else if( m_action == kDisable ) else if( m_action == kDisable )
event += ":disable"; event += ":disable";
else else
msg_Warn( getIntf(), "Unknown action type" ); msg_Warn( getIntf(), "unknown action type" );
return event; return event;
} }
......
...@@ -147,7 +147,7 @@ void Builder::addTheme( const BuilderData::Theme &rData ) ...@@ -147,7 +147,7 @@ void Builder::addTheme( const BuilderData::Theme &rData )
} }
else else
{ {
msg_Warn( getIntf(), "Invalid tooltip font: %s", msg_Warn( getIntf(), "invalid tooltip font: %s",
rData.m_tooltipfont.c_str() ); rData.m_tooltipfont.c_str() );
} }
} }
...@@ -173,7 +173,7 @@ void Builder::addSubBitmap( const BuilderData::SubBitmap &rData ) ...@@ -173,7 +173,7 @@ void Builder::addSubBitmap( const BuilderData::SubBitmap &rData )
{ {
if( m_pTheme->m_bitmaps.find( rData.m_id ) != m_pTheme->m_bitmaps.end() ) if( m_pTheme->m_bitmaps.find( rData.m_id ) != m_pTheme->m_bitmaps.end() )
{ {
msg_Dbg( getIntf(), "Bitmap %s already exists", rData.m_id.c_str() ); msg_Dbg( getIntf(), "bitmap %s already exists", rData.m_id.c_str() );
return; return;
} }
...@@ -191,7 +191,7 @@ void Builder::addSubBitmap( const BuilderData::SubBitmap &rData ) ...@@ -191,7 +191,7 @@ void Builder::addSubBitmap( const BuilderData::SubBitmap &rData )
{ {
// Invalid sub-bitmap // Invalid sub-bitmap
delete pBmp; delete pBmp;
msg_Warn( getIntf(), "SubBitmap %s ignored", rData.m_id.c_str() ); msg_Warn( getIntf(), "sub-bitmap %s ignored", rData.m_id.c_str() );
return; return;
} }
m_pTheme->m_bitmaps[rData.m_id] = GenericBitmapPtr( pBmp ); m_pTheme->m_bitmaps[rData.m_id] = GenericBitmapPtr( pBmp );
...@@ -202,7 +202,7 @@ void Builder::addBitmapFont( const BuilderData::BitmapFont &rData ) ...@@ -202,7 +202,7 @@ void Builder::addBitmapFont( const BuilderData::BitmapFont &rData )
{ {
if( m_pTheme->m_fonts.find( rData.m_id ) != m_pTheme->m_fonts.end() ) if( m_pTheme->m_fonts.find( rData.m_id ) != m_pTheme->m_fonts.end() )
{ {
msg_Dbg( getIntf(), "Font %s already exists", rData.m_id.c_str() ); msg_Dbg( getIntf(), "font %s already exists", rData.m_id.c_str() );
return; return;
} }
...@@ -282,14 +282,14 @@ void Builder::addMenuItem( const BuilderData::MenuItem &rData ) ...@@ -282,14 +282,14 @@ void Builder::addMenuItem( const BuilderData::MenuItem &rData )
Popup *pPopup = m_pTheme->getPopupById( rData.m_popupId ); Popup *pPopup = m_pTheme->getPopupById( rData.m_popupId );
if( pPopup == NULL ) if( pPopup == NULL )
{ {
msg_Err( getIntf(), "Unknown popup id: %s", rData.m_popupId.c_str() ); msg_Err( getIntf(), "unknown popup id: %s", rData.m_popupId.c_str() );
return; return;
} }
CmdGeneric *pCommand = parseAction( rData.m_action ); CmdGeneric *pCommand = parseAction( rData.m_action );
if( pCommand == NULL ) if( pCommand == NULL )
{ {
msg_Err( getIntf(), "Invalid action: %s", rData.m_action.c_str() ); msg_Err( getIntf(), "invalid action: %s", rData.m_action.c_str() );
return; return;
} }
...@@ -302,7 +302,7 @@ void Builder::addMenuSeparator( const BuilderData::MenuSeparator &rData ) ...@@ -302,7 +302,7 @@ void Builder::addMenuSeparator( const BuilderData::MenuSeparator &rData )
Popup *pPopup = m_pTheme->getPopupById( rData.m_popupId ); Popup *pPopup = m_pTheme->getPopupById( rData.m_popupId );
if( pPopup == NULL ) if( pPopup == NULL )
{ {
msg_Err( getIntf(), "Unknown popup id: %s", rData.m_popupId.c_str() ); msg_Err( getIntf(), "unknown popup id: %s", rData.m_popupId.c_str() );
return; return;
} }
...@@ -360,7 +360,7 @@ void Builder::addAnchor( const BuilderData::Anchor &rData ) ...@@ -360,7 +360,7 @@ void Builder::addAnchor( const BuilderData::Anchor &rData )
Bezier *pCurve = getPoints( rData.m_points.c_str() ); Bezier *pCurve = getPoints( rData.m_points.c_str() );
if( pCurve == NULL ) if( pCurve == NULL )
{ {
msg_Err( getIntf(), "Invalid format in tag points=\"%s\"", msg_Err( getIntf(), "invalid format in tag points=\"%s\"",
rData.m_points.c_str() ); rData.m_points.c_str() );
return; return;
} }
...@@ -395,7 +395,7 @@ void Builder::addButton( const BuilderData::Button &rData ) ...@@ -395,7 +395,7 @@ void Builder::addButton( const BuilderData::Button &rData )
CmdGeneric *pCommand = parseAction( rData.m_actionId ); CmdGeneric *pCommand = parseAction( rData.m_actionId );
if( pCommand == NULL ) if( pCommand == NULL )
{ {
msg_Err( getIntf(), "Invalid action: %s", rData.m_actionId.c_str() ); msg_Err( getIntf(), "invalid action: %s", rData.m_actionId.c_str() );
return; return;
} }
...@@ -452,14 +452,14 @@ void Builder::addCheckbox( const BuilderData::Checkbox &rData ) ...@@ -452,14 +452,14 @@ void Builder::addCheckbox( const BuilderData::Checkbox &rData )
CmdGeneric *pCommand1 = parseAction( rData.m_action1 ); CmdGeneric *pCommand1 = parseAction( rData.m_action1 );
if( pCommand1 == NULL ) if( pCommand1 == NULL )
{ {
msg_Err( getIntf(), "Invalid action: %s", rData.m_action1.c_str() ); msg_Err( getIntf(), "invalid action: %s", rData.m_action1.c_str() );
return; return;
} }
CmdGeneric *pCommand2 = parseAction( rData.m_action2 ); CmdGeneric *pCommand2 = parseAction( rData.m_action2 );
if( pCommand2 == NULL ) if( pCommand2 == NULL )
{ {
msg_Err( getIntf(), "Invalid action: %s", rData.m_action2.c_str() ); msg_Err( getIntf(), "invalid action: %s", rData.m_action2.c_str() );
return; return;
} }
...@@ -518,7 +518,7 @@ void Builder::addImage( const BuilderData::Image &rData ) ...@@ -518,7 +518,7 @@ void Builder::addImage( const BuilderData::Image &rData )
CmdGeneric *pCommand = parseAction( rData.m_action2Id ); CmdGeneric *pCommand = parseAction( rData.m_action2Id );
if( pCommand == NULL ) if( pCommand == NULL )
{ {
msg_Err( getIntf(), "Invalid action: %s", rData.m_action2Id.c_str() ); msg_Err( getIntf(), "invalid action: %s", rData.m_action2Id.c_str() );
return; return;
} }
...@@ -588,7 +588,7 @@ void Builder::addText( const BuilderData::Text &rData ) ...@@ -588,7 +588,7 @@ void Builder::addText( const BuilderData::Text &rData )
GenericFont *pFont = getFont( rData.m_fontId ); GenericFont *pFont = getFont( rData.m_fontId );
if( pFont == NULL ) if( pFont == NULL )
{ {
msg_Err( getIntf(), "Unknown font id: %s", rData.m_fontId.c_str() ); msg_Err( getIntf(), "unknown font id: %s", rData.m_fontId.c_str() );
return; return;
} }
...@@ -602,7 +602,7 @@ void Builder::addText( const BuilderData::Text &rData ) ...@@ -602,7 +602,7 @@ void Builder::addText( const BuilderData::Text &rData )
scrolling = CtrlText::kNone; scrolling = CtrlText::kNone;
else else
{ {
msg_Err( getIntf(), "Invalid scrolling mode: %s", msg_Err( getIntf(), "invalid scrolling mode: %s",
rData.m_scrolling.c_str() ); rData.m_scrolling.c_str() );
return; return;
} }
...@@ -617,7 +617,7 @@ void Builder::addText( const BuilderData::Text &rData ) ...@@ -617,7 +617,7 @@ void Builder::addText( const BuilderData::Text &rData )
alignment = CtrlText::kRight; alignment = CtrlText::kRight;
else else
{ {
msg_Err( getIntf(), "Invalid alignment: %s", msg_Err( getIntf(), "invalid alignment: %s",
rData.m_alignment.c_str() ); rData.m_alignment.c_str() );
return; return;
} }
...@@ -671,7 +671,7 @@ void Builder::addRadialSlider( const BuilderData::RadialSlider &rData ) ...@@ -671,7 +671,7 @@ void Builder::addRadialSlider( const BuilderData::RadialSlider &rData )
VarPercent *pVar = pInterpreter->getVarPercent( rData.m_value, m_pTheme ); VarPercent *pVar = pInterpreter->getVarPercent( rData.m_value, m_pTheme );
if( pVar == NULL ) if( pVar == NULL )
{ {
msg_Err( getIntf(), "Unknown slider value: %s", rData.m_value.c_str() ); msg_Err( getIntf(), "unknown slider value: %s", rData.m_value.c_str() );
return; return;
} }
...@@ -723,7 +723,7 @@ void Builder::addSlider( const BuilderData::Slider &rData ) ...@@ -723,7 +723,7 @@ void Builder::addSlider( const BuilderData::Slider &rData )
Bezier *pCurve = getPoints( rData.m_points.c_str() ); Bezier *pCurve = getPoints( rData.m_points.c_str() );
if( pCurve == NULL ) if( pCurve == NULL )
{ {
msg_Err( getIntf(), "Invalid format in tag points=\"%s\"", msg_Err( getIntf(), "invalid format in tag points=\"%s\"",
rData.m_points.c_str() ); rData.m_points.c_str() );
return; return;
} }
...@@ -738,7 +738,7 @@ void Builder::addSlider( const BuilderData::Slider &rData ) ...@@ -738,7 +738,7 @@ void Builder::addSlider( const BuilderData::Slider &rData )
VarPercent *pVar = pInterpreter->getVarPercent( rData.m_value, m_pTheme ); VarPercent *pVar = pInterpreter->getVarPercent( rData.m_value, m_pTheme );
if( pVar == NULL ) if( pVar == NULL )
{ {
msg_Err( getIntf(), "Unknown slider value: %s", rData.m_value.c_str() ); msg_Err( getIntf(), "unknown slider value: %s", rData.m_value.c_str() );
return; return;
} }
...@@ -799,7 +799,7 @@ void Builder::addList( const BuilderData::List &rData ) ...@@ -799,7 +799,7 @@ void Builder::addList( const BuilderData::List &rData )
GenericFont *pFont = getFont( rData.m_fontId ); GenericFont *pFont = getFont( rData.m_fontId );
if( pFont == NULL ) if( pFont == NULL )
{ {
msg_Err( getIntf(), "Unknown font id: %s", rData.m_fontId.c_str() ); msg_Err( getIntf(), "unknown font id: %s", rData.m_fontId.c_str() );
return; return;
} }
...@@ -808,7 +808,7 @@ void Builder::addList( const BuilderData::List &rData ) ...@@ -808,7 +808,7 @@ void Builder::addList( const BuilderData::List &rData )
VarList *pVar = pInterpreter->getVarList( rData.m_var, m_pTheme ); VarList *pVar = pInterpreter->getVarList( rData.m_var, m_pTheme );
if( pVar == NULL ) if( pVar == NULL )
{ {
msg_Err( getIntf(), "No such list variable: %s", rData.m_var.c_str() ); msg_Err( getIntf(), "no such list variable: %s", rData.m_var.c_str() );
return; return;
} }
...@@ -855,7 +855,7 @@ void Builder::addTree( const BuilderData::Tree &rData ) ...@@ -855,7 +855,7 @@ void Builder::addTree( const BuilderData::Tree &rData )
GenericFont *pFont = getFont( rData.m_fontId ); GenericFont *pFont = getFont( rData.m_fontId );
if( pFont == NULL ) if( pFont == NULL )
{ {
msg_Err( getIntf(), "Unknown font id: %s", rData.m_fontId.c_str() ); msg_Err( getIntf(), "unknown font id: %s", rData.m_fontId.c_str() );
return; return;
} }
...@@ -864,7 +864,7 @@ void Builder::addTree( const BuilderData::Tree &rData ) ...@@ -864,7 +864,7 @@ void Builder::addTree( const BuilderData::Tree &rData )
VarTree *pVar = pInterpreter->getVarTree( rData.m_var, m_pTheme ); VarTree *pVar = pInterpreter->getVarTree( rData.m_var, m_pTheme );
if( pVar == NULL ) if( pVar == NULL )
{ {
msg_Err( getIntf(), "No such list variable: %s", rData.m_var.c_str() ); msg_Err( getIntf(), "no such list variable: %s", rData.m_var.c_str() );
return; return;
} }
...@@ -1019,7 +1019,7 @@ GenericFont *Builder::getFont( const string &fontId ) ...@@ -1019,7 +1019,7 @@ GenericFont *Builder::getFont( const string &fontId )
} }
if( !pFont ) if( !pFont )
{ {
msg_Err( getIntf(), "Failed to open the default font" ); msg_Err( getIntf(), "failed to open the default font" );
} }
} }
return pFont; return pFont;
......
...@@ -193,16 +193,16 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme ) ...@@ -193,16 +193,16 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme )
GenericLayout *pLayout = pTheme->getLayoutById( layoutId ); GenericLayout *pLayout = pTheme->getLayoutById( layoutId );
if( !pWin ) if( !pWin )
{ {
msg_Err( getIntf(), "Unknown window (%s)", windowId.c_str() ); msg_Err( getIntf(), "unknown window (%s)", windowId.c_str() );
} }
else if( !pLayout ) else if( !pLayout )
{ {
msg_Err( getIntf(), "Unknown layout (%s)", layoutId.c_str() ); msg_Err( getIntf(), "unknown layout (%s)", layoutId.c_str() );
} }
// Check that the layout doesn't correspond to another window // Check that the layout doesn't correspond to another window
else if( pWin != pLayout->getWindow() ) else if( pWin != pLayout->getWindow() )
{ {
msg_Err( getIntf(), "Layout %s is not associated to window %s", msg_Err( getIntf(), "layout %s is not associated to window %s",
layoutId.c_str(), windowId.c_str() ); layoutId.c_str(), windowId.c_str() );
} }
else else
...@@ -230,7 +230,7 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme ) ...@@ -230,7 +230,7 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme )
} }
else else
{ {
msg_Err( getIntf(), "Unknown window or popup (%s)", msg_Err( getIntf(), "unknown window or popup (%s)",
windowId.c_str() ); windowId.c_str() );
} }
} }
...@@ -247,7 +247,7 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme ) ...@@ -247,7 +247,7 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme )
} }
else else
{ {
msg_Err( getIntf(), "Unknown window (%s)", windowId.c_str() ); msg_Err( getIntf(), "unknown window (%s)", windowId.c_str() );
} }
} }
...@@ -258,7 +258,7 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme ) ...@@ -258,7 +258,7 @@ CmdGeneric *Interpreter::parseAction( const string &rAction, Theme *pTheme )
} }
else else
{ {
msg_Warn( getIntf(), "Unknown action: %s", rAction.c_str() ); msg_Warn( getIntf(), "unknown action: %s", rAction.c_str() );
} }
return pCommand; return pCommand;
...@@ -284,7 +284,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -284,7 +284,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
// Get the 2 last variables on the stack // Get the 2 last variables on the stack
if( varStack.empty() ) if( varStack.empty() )
{ {
msg_Err( getIntf(), "Invalid boolean expression: %s", msg_Err( getIntf(), "invalid boolean expression: %s",
rName.c_str()); rName.c_str());
return NULL; return NULL;
} }
...@@ -292,7 +292,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -292,7 +292,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
varStack.pop_back(); varStack.pop_back();
if( varStack.empty() ) if( varStack.empty() )
{ {
msg_Err( getIntf(), "Invalid boolean expression: %s", msg_Err( getIntf(), "invalid boolean expression: %s",
rName.c_str()); rName.c_str());
return NULL; return NULL;
} }
...@@ -310,7 +310,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -310,7 +310,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
// Get the 2 last variables on the stack // Get the 2 last variables on the stack
if( varStack.empty() ) if( varStack.empty() )
{ {
msg_Err( getIntf(), "Invalid boolean expression: %s", msg_Err( getIntf(), "invalid boolean expression: %s",
rName.c_str()); rName.c_str());
return NULL; return NULL;
} }
...@@ -318,7 +318,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -318,7 +318,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
varStack.pop_back(); varStack.pop_back();
if( varStack.empty() ) if( varStack.empty() )
{ {
msg_Err( getIntf(), "Invalid boolean expression: %s", msg_Err( getIntf(), "invalid boolean expression: %s",
rName.c_str()); rName.c_str());
return NULL; return NULL;
} }
...@@ -336,7 +336,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -336,7 +336,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
// Get the last variable on the stack // Get the last variable on the stack
if( varStack.empty() ) if( varStack.empty() )
{ {
msg_Err( getIntf(), "Invalid boolean expression: %s", msg_Err( getIntf(), "invalid boolean expression: %s",
rName.c_str()); rName.c_str());
return NULL; return NULL;
} }
...@@ -361,7 +361,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -361,7 +361,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
} }
else else
{ {
msg_Err( getIntf(), "Unknown window (%s)", windowId.c_str() ); msg_Err( getIntf(), "unknown window (%s)", windowId.c_str() );
return NULL; return NULL;
} }
} }
...@@ -371,7 +371,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -371,7 +371,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
VarBool *pVar = (VarBool*)pVarManager->getVar( token, "bool" ); VarBool *pVar = (VarBool*)pVarManager->getVar( token, "bool" );
if( !pVar ) if( !pVar )
{ {
msg_Err( getIntf(), "Cannot resolve boolean variable: %s", msg_Err( getIntf(), "cannot resolve boolean variable: %s",
token.c_str()); token.c_str());
return NULL; return NULL;
} }
...@@ -384,7 +384,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme ) ...@@ -384,7 +384,7 @@ VarBool *Interpreter::getVarBool( const string &rName, Theme *pTheme )
// The stack should contain a single variable // The stack should contain a single variable
if( varStack.size() != 1 ) if( varStack.size() != 1 )
{ {
msg_Err( getIntf(), "Invalid boolean expression: %s", rName.c_str() ); msg_Err( getIntf(), "invalid boolean expression: %s", rName.c_str() );
return NULL; return NULL;
} }
return varStack.back(); return varStack.back();
......
...@@ -54,7 +54,7 @@ void SkinParser::handleBeginElement( const string &rName, AttrList_t &attr ) ...@@ -54,7 +54,7 @@ void SkinParser::handleBeginElement( const string &rName, AttrList_t &attr )
#define RequireDefault( a ) \ #define RequireDefault( a ) \
if( attr.find(a) == attr.end() ) \ if( attr.find(a) == attr.end() ) \
{ \ { \
msg_Err( getIntf(), "Bad theme (element: %s, missing attribute: %s)", \ msg_Err( getIntf(), "bad theme (element: %s, missing attribute: %s)", \
rName.c_str(), a ); \ rName.c_str(), a ); \
m_errors = true; return; \ m_errors = true; return; \
} }
...@@ -65,7 +65,7 @@ void SkinParser::handleBeginElement( const string &rName, AttrList_t &attr ) ...@@ -65,7 +65,7 @@ void SkinParser::handleBeginElement( const string &rName, AttrList_t &attr )
OSFactory *pFactory = OSFactory::instance( getIntf() ); OSFactory *pFactory = OSFactory::instance( getIntf() );
string fullPath = m_path + pFactory->getDirSeparator() + attr["file"]; string fullPath = m_path + pFactory->getDirSeparator() + attr["file"];
msg_Dbg( getIntf(), "Opening included XML file: %s", fullPath.c_str() ); msg_Dbg( getIntf(), "opening included XML file: %s", fullPath.c_str() );
// FIXME: We do not use the DTD to validate the included XML file, // FIXME: We do not use the DTD to validate the included XML file,
// as the parser seems to dislike it otherwise... // as the parser seems to dislike it otherwise...
SkinParser subParser( getIntf(), fullPath.c_str(), false, m_pData ); SkinParser subParser( getIntf(), fullPath.c_str(), false, m_pData );
...@@ -484,7 +484,7 @@ void SkinParser::handleBeginElement( const string &rName, AttrList_t &attr ) ...@@ -484,7 +484,7 @@ void SkinParser::handleBeginElement( const string &rName, AttrList_t &attr )
// Check the version // Check the version
if( strcmp( attr["version"], SKINS_DTD_VERSION ) ) if( strcmp( attr["version"], SKINS_DTD_VERSION ) )
{ {
msg_Err( getIntf(), "Bad theme version : %s (you need version %s)", msg_Err( getIntf(), "bad theme version : %s (you need version %s)",
attr["version"], SKINS_DTD_VERSION ); attr["version"], SKINS_DTD_VERSION );
m_errors = true; m_errors = true;
return; return;
...@@ -592,14 +592,14 @@ int SkinParser::convertInRange( const char *value, int minValue, int maxValue, ...@@ -592,14 +592,14 @@ int SkinParser::convertInRange( const char *value, int minValue, int maxValue,
if( intValue < minValue ) if( intValue < minValue )
{ {
msg_Warn( getIntf(), "Value of \"%s\" attribute (%i) is out of the " msg_Warn( getIntf(), "value of \"%s\" attribute (%i) is out of the "
"expected range [%i, %i], using %i instead", "expected range [%i, %i], using %i instead",
rAttribute.c_str(), intValue, minValue, maxValue, minValue ); rAttribute.c_str(), intValue, minValue, maxValue, minValue );
return minValue; return minValue;
} }
else if( intValue > maxValue ) else if( intValue > maxValue )
{ {
msg_Warn( getIntf(), "Value of \"%s\" attribute (%i) is out of the " msg_Warn( getIntf(), "value of \"%s\" attribute (%i) is out of the "
"expected range [%i, %i], using %i instead", "expected range [%i, %i], using %i instead",
rAttribute.c_str(), intValue, minValue, maxValue, maxValue ); rAttribute.c_str(), intValue, minValue, maxValue, maxValue );
return maxValue; return maxValue;
...@@ -633,7 +633,7 @@ const string SkinParser::uniqueId( const string &id ) ...@@ -633,7 +633,7 @@ const string SkinParser::uniqueId( const string &id )
// The id was already used // The id was already used
if( id != "none" ) if( id != "none" )
{ {
msg_Warn( getIntf(), "Non unique id: %s", id.c_str() ); msg_Warn( getIntf(), "non-unique id: %s", id.c_str() );
} }
newId = generateId(); newId = generateId();
} }
......
...@@ -41,7 +41,7 @@ XMLParser::XMLParser( intf_thread_t *pIntf, const string &rFileName, ...@@ -41,7 +41,7 @@ XMLParser::XMLParser( intf_thread_t *pIntf, const string &rFileName,
m_pXML = xml_Create( pIntf ); m_pXML = xml_Create( pIntf );
if( !m_pXML ) if( !m_pXML )
{ {
msg_Err( getIntf(), "Failed to open XML parser" ); msg_Err( getIntf(), "failed to open XML parser" );
return; return;
} }
...@@ -55,14 +55,14 @@ XMLParser::XMLParser( intf_thread_t *pIntf, const string &rFileName, ...@@ -55,14 +55,14 @@ XMLParser::XMLParser( intf_thread_t *pIntf, const string &rFileName,
m_pStream = stream_UrlNew( pIntf, rFileName.c_str() ); m_pStream = stream_UrlNew( pIntf, rFileName.c_str() );
if( !m_pStream ) if( !m_pStream )
{ {
msg_Err( getIntf(), "Failed to open %s for reading", msg_Err( getIntf(), "failed to open %s for reading",
rFileName.c_str() ); rFileName.c_str() );
return; return;
} }
m_pReader = xml_ReaderCreate( m_pXML, m_pStream ); m_pReader = xml_ReaderCreate( m_pXML, m_pStream );
if( !m_pReader ) if( !m_pReader )
{ {
msg_Err( getIntf(), "Failed to open %s for parsing", msg_Err( getIntf(), "failed to open %s for parsing",
rFileName.c_str() ); rFileName.c_str() );
return; return;
} }
...@@ -115,7 +115,7 @@ void XMLParser::LoadCatalog() ...@@ -115,7 +115,7 @@ void XMLParser::LoadCatalog()
if( !stat( path.c_str(), &statBuf ) ) if( !stat( path.c_str(), &statBuf ) )
{ {
// DTD found // DTD found
msg_Dbg( getIntf(), "Using DTD %s", path.c_str() ); msg_Dbg( getIntf(), "using DTD %s", path.c_str() );
// Add an entry in the default catalog // Add an entry in the default catalog
xml_CatalogAdd( m_pXML, "public", xml_CatalogAdd( m_pXML, "public",
...@@ -126,7 +126,7 @@ void XMLParser::LoadCatalog() ...@@ -126,7 +126,7 @@ void XMLParser::LoadCatalog()
} }
if( it == resPath.end() ) if( it == resPath.end() )
{ {
msg_Err( getIntf(), "Cannot find the skins DTD !"); msg_Err( getIntf(), "cannot find the skins DTD");
} }
#endif #endif
} }
......
...@@ -170,7 +170,7 @@ bool Dialogs::init() ...@@ -170,7 +170,7 @@ bool Dialogs::init()
m_pModule = module_Need( m_pProvider, "dialogs provider", NULL, 0 ); m_pModule = module_Need( m_pProvider, "dialogs provider", NULL, 0 );
if( m_pModule == NULL ) if( m_pModule == NULL )
{ {
msg_Err( getIntf(), "No suitable dialogs provider found (hint: compile the wxWidgets plugin, and make sure it is loaded properly)" ); msg_Err( getIntf(), "no suitable dialogs provider found (hint: compile the wxWidgets plugin, and make sure it is loaded properly)" );
vlc_object_destroy( m_pProvider ); vlc_object_destroy( m_pProvider );
m_pProvider = NULL; m_pProvider = NULL;
return false; return false;
......
...@@ -68,7 +68,7 @@ bool FT2Font::init() ...@@ -68,7 +68,7 @@ bool FT2Font::init()
// Initalise libfreetype // Initalise libfreetype
if( FT_Init_FreeType( &m_lib ) ) if( FT_Init_FreeType( &m_lib ) )
{ {
msg_Err( getIntf(), "Failed to initalize libfreetype" ); msg_Err( getIntf(), "failed to initalize freetype" );
return false; return false;
} }
...@@ -76,11 +76,11 @@ bool FT2Font::init() ...@@ -76,11 +76,11 @@ bool FT2Font::init()
FILE *file = fopen( m_name.c_str(), "rb" ); FILE *file = fopen( m_name.c_str(), "rb" );
if( file ) if( file )
{ {
msg_Dbg( getIntf(), "Loading font %s", m_name.c_str() ); msg_Dbg( getIntf(), "loading font %s", m_name.c_str() );
} }
else else
{ {
msg_Dbg( getIntf(), "Unable to open the font %s", m_name.c_str() ); msg_Dbg( getIntf(), "unable to open the font %s", m_name.c_str() );
return false; return false;
} }
// Get the file size // Get the file size
...@@ -91,7 +91,7 @@ bool FT2Font::init() ...@@ -91,7 +91,7 @@ bool FT2Font::init()
m_buffer = malloc( size ); m_buffer = malloc( size );
if( !m_buffer ) if( !m_buffer )
{ {
msg_Err( getIntf(), "Not enough memory for the font %s", msg_Err( getIntf(), "not enough memory for the font %s",
m_name.c_str() ); m_name.c_str() );
return false; return false;
} }
...@@ -104,26 +104,26 @@ bool FT2Font::init() ...@@ -104,26 +104,26 @@ bool FT2Font::init()
&m_face ); &m_face );
if ( err == FT_Err_Unknown_File_Format ) if ( err == FT_Err_Unknown_File_Format )
{ {
msg_Err( getIntf(), "Unsupported font format (%s)", m_name.c_str() ); msg_Err( getIntf(), "unsupported font format (%s)", m_name.c_str() );
return false; return false;
} }
else if ( err ) else if ( err )
{ {
msg_Err( getIntf(), "Error opening font (%s)", m_name.c_str() ); msg_Err( getIntf(), "error opening font (%s)", m_name.c_str() );
return false; return false;
} }
// Select the charset // Select the charset
if( FT_Select_Charmap( m_face, ft_encoding_unicode ) ) if( FT_Select_Charmap( m_face, ft_encoding_unicode ) )
{ {
msg_Err( getIntf(), "Font has no UNICODE table (%s)", m_name.c_str() ); msg_Err( getIntf(), "font has no UNICODE table (%s)", m_name.c_str() );
return false; return false;
} }
// Set the pixel size // Set the pixel size
if( FT_Set_Pixel_Sizes( m_face, 0, m_size ) ) if( FT_Set_Pixel_Sizes( m_face, 0, m_size ) )
{ {
msg_Warn( getIntf(), "Cannot set a pixel size of %d (%s)", m_size, msg_Warn( getIntf(), "cannot set a pixel size of %d (%s)", m_size,
m_name.c_str() ); m_name.c_str() );
} }
......
...@@ -110,7 +110,7 @@ void GenericLayout::addControl( CtrlGeneric *pControl, ...@@ -110,7 +110,7 @@ void GenericLayout::addControl( CtrlGeneric *pControl,
} }
else else
{ {
msg_Dbg( getIntf(), "Adding NULL control in the layout" ); msg_Dbg( getIntf(), "adding NULL control in the layout" );
} }
} }
......
...@@ -80,7 +80,7 @@ void Popup::handleEvent( const EvtMenu &rEvent ) ...@@ -80,7 +80,7 @@ void Popup::handleEvent( const EvtMenu &rEvent )
else else
{ {
// Should never happen // Should never happen
msg_Warn( getIntf(), "Problem in the popup implementation" ); msg_Warn( getIntf(), "problem in the popup implementation" );
} }
} }
...@@ -109,35 +109,35 @@ static int Open( vlc_object_t *p_this ) ...@@ -109,35 +109,35 @@ static int Open( vlc_object_t *p_this )
// Initialize singletons // Initialize singletons
if( OSFactory::instance( p_intf ) == NULL ) if( OSFactory::instance( p_intf ) == NULL )
{ {
msg_Err( p_intf, "Cannot initialize OSFactory" ); msg_Err( p_intf, "cannot initialize OSFactory" );
vlc_object_release( p_intf->p_sys->p_playlist ); vlc_object_release( p_intf->p_sys->p_playlist );
msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub ); msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub );
return VLC_EGENERIC; return VLC_EGENERIC;
} }
if( AsyncQueue::instance( p_intf ) == NULL ) if( AsyncQueue::instance( p_intf ) == NULL )
{ {
msg_Err( p_intf, "Cannot initialize AsyncQueue" ); msg_Err( p_intf, "cannot initialize AsyncQueue" );
vlc_object_release( p_intf->p_sys->p_playlist ); vlc_object_release( p_intf->p_sys->p_playlist );
msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub ); msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub );
return VLC_EGENERIC; return VLC_EGENERIC;
} }
if( Interpreter::instance( p_intf ) == NULL ) if( Interpreter::instance( p_intf ) == NULL )
{ {
msg_Err( p_intf, "Cannot instanciate Interpreter" ); msg_Err( p_intf, "cannot instanciate Interpreter" );
vlc_object_release( p_intf->p_sys->p_playlist ); vlc_object_release( p_intf->p_sys->p_playlist );
msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub ); msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub );
return VLC_EGENERIC; return VLC_EGENERIC;
} }
if( VarManager::instance( p_intf ) == NULL ) if( VarManager::instance( p_intf ) == NULL )
{ {
msg_Err( p_intf, "Cannot instanciate VarManager" ); msg_Err( p_intf, "cannot instanciate VarManager" );
vlc_object_release( p_intf->p_sys->p_playlist ); vlc_object_release( p_intf->p_sys->p_playlist );
msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub ); msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub );
return VLC_EGENERIC; return VLC_EGENERIC;
} }
if( VlcProc::instance( p_intf ) == NULL ) if( VlcProc::instance( p_intf ) == NULL )
{ {
msg_Err( p_intf, "Cannot initialize VLCProc" ); msg_Err( p_intf, "cannot initialize VLCProc" );
vlc_object_release( p_intf->p_sys->p_playlist ); vlc_object_release( p_intf->p_sys->p_playlist );
msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub ); msg_Unsubscribe( p_intf, p_intf->p_sys->p_sub );
return VLC_EGENERIC; return VLC_EGENERIC;
...@@ -223,7 +223,7 @@ static void Run( intf_thread_t *p_intf ) ...@@ -223,7 +223,7 @@ static void Run( intf_thread_t *p_intf )
AsyncQueue *pQueue = AsyncQueue::instance( p_intf ); AsyncQueue *pQueue = AsyncQueue::instance( p_intf );
pQueue->push( CmdGenericPtr( pCmd ) ); pQueue->push( CmdGenericPtr( pCmd ) );
msg_Err( p_intf, msg_Err( p_intf,
"Cannot show the \"open skin\" dialog: exiting..."); "cannot show the \"open skin\" dialog: exiting...");
} }
} }
} }
......
...@@ -41,7 +41,7 @@ Theme::~Theme() ...@@ -41,7 +41,7 @@ Theme::~Theme()
void Theme::loadConfig() void Theme::loadConfig()
{ {
msg_Dbg( getIntf(), "Loading theme configuration"); msg_Dbg( getIntf(), "loading theme configuration");
// Get config from vlcrc file // Get config from vlcrc file
char *save = config_GetPsz( getIntf(), "skins2-config" ); char *save = config_GetPsz( getIntf(), "skins2-config" );
...@@ -88,7 +88,7 @@ void Theme::loadConfig() ...@@ -88,7 +88,7 @@ void Theme::loadConfig()
void Theme::saveConfig() void Theme::saveConfig()
{ {
msg_Dbg( getIntf(), "Saving theme configuration"); msg_Dbg( getIntf(), "saving theme configuration");
// Initialize char where config is stored // Initialize char where config is stored
char *save = new char[400]; char *save = new char[400];
......
...@@ -163,7 +163,7 @@ bool ThemeLoader::extractZip( const string &zipFile, const string &rootDir ) ...@@ -163,7 +163,7 @@ bool ThemeLoader::extractZip( const string &zipFile, const string &rootDir )
{ {
if( !extractFileInZip( file, rootDir ) ) if( !extractFileInZip( file, rootDir ) )
{ {
msg_Warn( getIntf(), "Error while unzipping %s", msg_Warn( getIntf(), "error while unzipping %s",
zipFile.c_str() ); zipFile.c_str() );
unzClose( file ); unzClose( file );
return false; return false;
...@@ -174,7 +174,7 @@ bool ThemeLoader::extractZip( const string &zipFile, const string &rootDir ) ...@@ -174,7 +174,7 @@ bool ThemeLoader::extractZip( const string &zipFile, const string &rootDir )
// Go the next file in the archive // Go the next file in the archive
if( unzGoToNextFile( file ) !=UNZ_OK ) if( unzGoToNextFile( file ) !=UNZ_OK )
{ {
msg_Warn( getIntf(), "Error while unzipping %s", msg_Warn( getIntf(), "error while unzipping %s",
zipFile.c_str() ); zipFile.c_str() );
unzClose( file ); unzClose( file );
return false; return false;
...@@ -209,7 +209,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir ) ...@@ -209,7 +209,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
void *pBuffer = malloc( ZIP_BUFFER_SIZE ); void *pBuffer = malloc( ZIP_BUFFER_SIZE );
if( !pBuffer ) if( !pBuffer )
{ {
msg_Err( getIntf(), "Failed to allocate memory" ); msg_Err( getIntf(), "failed to allocate memory" );
return false; return false;
} }
...@@ -232,7 +232,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir ) ...@@ -232,7 +232,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
FILE *fout = fopen( fullPath.c_str(), "wb" ); FILE *fout = fopen( fullPath.c_str(), "wb" );
if( fout == NULL ) if( fout == NULL )
{ {
msg_Err( getIntf(), "Error opening %s", fullPath.c_str() ); msg_Err( getIntf(), "error opening %s", fullPath.c_str() );
free( pBuffer ); free( pBuffer );
return false; return false;
} }
...@@ -244,7 +244,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir ) ...@@ -244,7 +244,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
n = unzReadCurrentFile( file, pBuffer, ZIP_BUFFER_SIZE ); n = unzReadCurrentFile( file, pBuffer, ZIP_BUFFER_SIZE );
if( n < 0 ) if( n < 0 )
{ {
msg_Err( getIntf(), "Error while reading zip file" ); msg_Err( getIntf(), "error while reading zip file" );
free( pBuffer ); free( pBuffer );
return false; return false;
} }
...@@ -252,7 +252,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir ) ...@@ -252,7 +252,7 @@ bool ThemeLoader::extractFileInZip( unzFile file, const string &rootDir )
{ {
if( fwrite( pBuffer, n , 1, fout) != 1 ) if( fwrite( pBuffer, n , 1, fout) != 1 )
{ {
msg_Err( getIntf(), "Error while writing %s", msg_Err( getIntf(), "error while writing %s",
fullPath.c_str() ); fullPath.c_str() );
free( pBuffer ); free( pBuffer );
return false; return false;
...@@ -303,7 +303,7 @@ bool ThemeLoader::extract( const string &fileName ) ...@@ -303,7 +303,7 @@ bool ThemeLoader::extract( const string &fileName )
string mainBmp; string mainBmp;
if( findFile( tempPath, "main.bmp", mainBmp ) ) if( findFile( tempPath, "main.bmp", mainBmp ) )
{ {
msg_Dbg( getIntf(), "Try to load a winamp2 skin" ); msg_Dbg( getIntf(), "trying to load a winamp2 skin" );
path = getFilePath( mainBmp ); path = getFilePath( mainBmp );
// Look for winamp2.xml in the resource path // Look for winamp2.xml in the resource path
...@@ -322,13 +322,13 @@ bool ThemeLoader::extract( const string &fileName ) ...@@ -322,13 +322,13 @@ bool ThemeLoader::extract( const string &fileName )
// Parse the XML file // Parse the XML file
if (! parse( path, xmlFile ) ) if (! parse( path, xmlFile ) )
{ {
msg_Err( getIntf(), "Error while parsing %s", xmlFile.c_str() ); msg_Err( getIntf(), "error while parsing %s", xmlFile.c_str() );
result = false; result = false;
} }
} }
else else
{ {
msg_Err( getIntf(), "No XML found in theme %s", fileName.c_str() ); msg_Err( getIntf(), "no XML found in theme %s", fileName.c_str() );
result = false; result = false;
} }
...@@ -348,13 +348,13 @@ void ThemeLoader::deleteTempFiles( const string &path ) ...@@ -348,13 +348,13 @@ void ThemeLoader::deleteTempFiles( const string &path )
bool ThemeLoader::parse( const string &path, const string &xmlFile ) bool ThemeLoader::parse( const string &path, const string &xmlFile )
{ {
// File loaded // File loaded
msg_Dbg( getIntf(), "Using skin file: %s", xmlFile.c_str() ); msg_Dbg( getIntf(), "using skin file: %s", xmlFile.c_str() );
// Start the parser // Start the parser
SkinParser parser( getIntf(), xmlFile, path ); SkinParser parser( getIntf(), xmlFile, path );
if( ! parser.parse() ) if( ! parser.parse() )
{ {
msg_Err( getIntf(), "Failed to parse %s", xmlFile.c_str() ); msg_Err( getIntf(), "failed to parse %s", xmlFile.c_str() );
return false; return false;
} }
...@@ -418,7 +418,7 @@ bool ThemeLoader::findFile( const string &rootDir, const string &rFileName, ...@@ -418,7 +418,7 @@ bool ThemeLoader::findFile( const string &rootDir, const string &rFileName,
if( pCurrDir == NULL ) if( pCurrDir == NULL )
{ {
// An error occurred // An error occurred
msg_Dbg( getIntf(), "Cannot open directory %s", rootDir.c_str() ); msg_Dbg( getIntf(), "cannot open directory %s", rootDir.c_str() );
return false; return false;
} }
......
...@@ -109,7 +109,7 @@ void ThemeRepository::parseDirectory( const string &rDir ) ...@@ -109,7 +109,7 @@ void ThemeRepository::parseDirectory( const string &rDir )
if( pDir == NULL ) if( pDir == NULL )
{ {
// An error occurred // An error occurred
msg_Dbg( getIntf(), "Cannot open directory %s", rDir.c_str() ); msg_Dbg( getIntf(), "cannot open directory %s", rDir.c_str() );
return; return;
} }
......
...@@ -98,7 +98,7 @@ void TopWindow::processEvent( EvtMenu &rEvtMenu ) ...@@ -98,7 +98,7 @@ void TopWindow::processEvent( EvtMenu &rEvtMenu )
// We should never receive a menu event when there is no active popup! // We should never receive a menu event when there is no active popup!
if( pPopup == NULL ) if( pPopup == NULL )
{ {
msg_Warn( getIntf(), "Unexpected menu event, ignoring" ); msg_Warn( getIntf(), "unexpected menu event, ignoring" );
return; return;
} }
...@@ -380,7 +380,7 @@ void TopWindow::onControlRelease( const CtrlGeneric &rCtrl ) ...@@ -380,7 +380,7 @@ void TopWindow::onControlRelease( const CtrlGeneric &rCtrl )
} }
else else
{ {
msg_Dbg( getIntf(), "Control had not captured the mouse" ); msg_Dbg( getIntf(), "control had not captured the mouse" );
} }
// Send an enter event to the control under the mouse, if it doesn't // Send an enter event to the control under the mouse, if it doesn't
...@@ -451,7 +451,7 @@ CtrlGeneric *TopWindow::findHitControl( int xPos, int yPos ) ...@@ -451,7 +451,7 @@ CtrlGeneric *TopWindow::findHitControl( int xPos, int yPos )
} }
else else
{ {
msg_Dbg( getIntf(), "Control at NULL position" ); msg_Dbg( getIntf(), "control at NULL position" );
} }
} }
......
...@@ -115,7 +115,7 @@ Variable *VarManager::getVar( const string &rName, const string &rType ) ...@@ -115,7 +115,7 @@ Variable *VarManager::getVar( const string &rName, const string &rType )
// Check the variable type // Check the variable type
if( pVar->getType() != rType ) if( pVar->getType() != rType )
{ {
msg_Warn( getIntf(), "Variable %s has incorrect type (%s instead" msg_Warn( getIntf(), "variable %s has incorrect type (%s instead"
" of (%s)", rName.c_str(), pVar->getType().c_str(), " of (%s)", rName.c_str(), pVar->getType().c_str(),
rType.c_str() ); rType.c_str() );
return NULL; return NULL;
......
...@@ -381,7 +381,7 @@ void WindowManager::createTooltip( const GenericFont &rTipFont ) ...@@ -381,7 +381,7 @@ void WindowManager::createTooltip( const GenericFont &rTipFont )
} }
else else
{ {
msg_Warn( getIntf(), "Tooltip already created!"); msg_Warn( getIntf(), "tooltip already created!");
} }
} }
......
...@@ -71,7 +71,7 @@ UString::UString( intf_thread_t *pIntf, const char *pString ): ...@@ -71,7 +71,7 @@ UString::UString( intf_thread_t *pIntf, const char *pString ):
} }
if( !pCur || *pCur ) if( !pCur || *pCur )
{ {
msg_Err( pIntf, "Invalid UTF8 string: %s", pString ); msg_Err( pIntf, "invalid UTF8 string: %s", pString );
m_length = 0; m_length = 0;
m_pString = NULL; m_pString = NULL;
return; return;
...@@ -315,7 +315,7 @@ UString UString::substr( uint32_t position, uint32_t n) const ...@@ -315,7 +315,7 @@ UString UString::substr( uint32_t position, uint32_t n) const
UString tmp( getIntf(), "" ); UString tmp( getIntf(), "" );
if( position > size() ) if( position > size() )
{ {
msg_Err( getIntf(), "Invalid position in UString::substr()" ); msg_Err( getIntf(), "invalid position in UString::substr()" );
return tmp; return tmp;
} }
tmp.m_length = (n < size() - position) ? n : size() - position; tmp.m_length = (n < size() - position) ? n : size() - position;
......
...@@ -40,12 +40,12 @@ Playtree::Playtree( intf_thread_t *pIntf ): VarTree( pIntf ) ...@@ -40,12 +40,12 @@ Playtree::Playtree( intf_thread_t *pIntf ): VarTree( pIntf )
char *pCharset; char *pCharset;
vlc_current_charset( &pCharset ); vlc_current_charset( &pCharset );
iconvHandle = vlc_iconv_open( "UTF-8", pCharset ); iconvHandle = vlc_iconv_open( "UTF-8", pCharset );
msg_Dbg( pIntf, "Using character encoding: %s", pCharset ); msg_Dbg( pIntf, "using character encoding: %s", pCharset );
free( pCharset ); free( pCharset );
if( iconvHandle == (vlc_iconv_t) - 1 ) if( iconvHandle == (vlc_iconv_t) - 1 )
{ {
msg_Warn( pIntf, "Unable to do requested conversion" ); msg_Warn( pIntf, "unable to do requested conversion" );
} }
buildTree(); buildTree();
...@@ -149,7 +149,7 @@ void Playtree::onUpdateItem( int id ) ...@@ -149,7 +149,7 @@ void Playtree::onUpdateItem( int id )
} }
else else
{ {
msg_Warn(getIntf(), "Cannot find node with id %d", id ); msg_Warn(getIntf(), "cannot find node with id %d", id );
} }
descr.i_type = 0; descr.i_type = 0;
notify( &descr ); notify( &descr );
......
...@@ -120,7 +120,7 @@ bool Win32Factory::init() ...@@ -120,7 +120,7 @@ bool Win32Factory::init()
// then fine, otherwise return with an error. // then fine, otherwise return with an error.
if( !GetClassInfo( m_hInst, _T("SkinWindowClass"), &wndclass ) ) if( !GetClassInfo( m_hInst, _T("SkinWindowClass"), &wndclass ) )
{ {
msg_Err( getIntf(), "Cannot register window class" ); msg_Err( getIntf(), "cannot register window class" );
return false; return false;
} }
} }
...@@ -131,7 +131,7 @@ bool Win32Factory::init() ...@@ -131,7 +131,7 @@ bool Win32Factory::init()
-200, -200, 0, 0, 0, 0, m_hInst, 0 ); -200, -200, 0, 0, 0, 0, m_hInst, 0 );
if( m_hParentWindow == NULL ) if( m_hParentWindow == NULL )
{ {
msg_Err( getIntf(), "Cannot create parent window" ); msg_Err( getIntf(), "cannot create parent window" );
return false; return false;
} }
...@@ -157,7 +157,7 @@ bool Win32Factory::init() ...@@ -157,7 +157,7 @@ bool Win32Factory::init()
GetProcAddress( m_hMsimg32, _T("TransparentBlt") ) ) ) GetProcAddress( m_hMsimg32, _T("TransparentBlt") ) ) )
{ {
TransparentBlt = NULL; TransparentBlt = NULL;
msg_Dbg( getIntf(), "Couldn't find TransparentBlt(), " msg_Dbg( getIntf(), "couldn't find TransparentBlt(), "
"falling back to BitBlt()" ); "falling back to BitBlt()" );
} }
if( !m_hMsimg32 || if( !m_hMsimg32 ||
...@@ -167,7 +167,7 @@ bool Win32Factory::init() ...@@ -167,7 +167,7 @@ bool Win32Factory::init()
GetProcAddress( m_hMsimg32, _T("AlphaBlend") ) ) ) GetProcAddress( m_hMsimg32, _T("AlphaBlend") ) ) )
{ {
AlphaBlend = NULL; AlphaBlend = NULL;
msg_Dbg( getIntf(), "Couldn't find AlphaBlend()" ); msg_Dbg( getIntf(), "couldn't find AlphaBlend()" );
} }
// Idem for user32.dll and SetLayeredWindowAttributes() // Idem for user32.dll and SetLayeredWindowAttributes()
...@@ -178,7 +178,7 @@ bool Win32Factory::init() ...@@ -178,7 +178,7 @@ bool Win32Factory::init()
GetProcAddress( m_hUser32, _T("SetLayeredWindowAttributes") ) ) ) GetProcAddress( m_hUser32, _T("SetLayeredWindowAttributes") ) ) )
{ {
SetLayeredWindowAttributes = NULL; SetLayeredWindowAttributes = NULL;
msg_Dbg( getIntf(), "Couldn't find SetLayeredWindowAttributes()" ); msg_Dbg( getIntf(), "couldn't find SetLayeredWindowAttributes()" );
} }
// Initialize the resource path // Initialize the resource path
...@@ -267,7 +267,7 @@ OSPopup *Win32Factory::createOSPopup() ...@@ -267,7 +267,7 @@ OSPopup *Win32Factory::createOSPopup()
// GenericWindow (we cannot eevn access the OSWindow). // GenericWindow (we cannot eevn access the OSWindow).
if( m_windowMap.begin() == m_windowMap.end() ) if( m_windowMap.begin() == m_windowMap.end() )
{ {
msg_Err( getIntf(), "No window has been created before the popup!" ); msg_Err( getIntf(), "no window has been created before the popup!" );
return NULL; return NULL;
} }
......
...@@ -40,7 +40,7 @@ Win32Tooltip::Win32Tooltip( intf_thread_t *pIntf, HINSTANCE hInst, ...@@ -40,7 +40,7 @@ Win32Tooltip::Win32Tooltip( intf_thread_t *pIntf, HINSTANCE hInst,
if( !m_hWnd ) if( !m_hWnd )
{ {
msg_Err( getIntf(), "CreateWindow failed" ); msg_Err( getIntf(), "createWindow failed" );
return; return;
} }
} }
......
...@@ -183,7 +183,7 @@ X11Display::X11Display( intf_thread_t *pIntf ): SkinObject( pIntf ), ...@@ -183,7 +183,7 @@ X11Display::X11Display( intf_thread_t *pIntf ): SkinObject( pIntf ),
break; break;
default: default:
msg_Err( getIntf(), "Unsupported depth: %d bpp\n", depth ); msg_Err( getIntf(), "unsupported depth: %d bpp\n", depth );
m_pDisplay = NULL; m_pDisplay = NULL;
break; break;
} }
......
...@@ -49,7 +49,7 @@ X11Graphics::X11Graphics( intf_thread_t *pIntf, X11Display &rDisplay, ...@@ -49,7 +49,7 @@ X11Graphics::X11Graphics( intf_thread_t *pIntf, X11Display &rDisplay,
{ {
// Avoid a X11 Bad Value error // Avoid a X11 Bad Value error
width = height = 1; width = height = 1;
msg_Err( getIntf(), "Invalid image size (null width or height)" ); msg_Err( getIntf(), "invalid image size (null width or height)" );
} }
// Create a pixmap // Create a pixmap
...@@ -138,7 +138,7 @@ void X11Graphics::drawBitmap( const GenericBitmap &rBitmap, int xSrc, ...@@ -138,7 +138,7 @@ void X11Graphics::drawBitmap( const GenericBitmap &rBitmap, int xSrc,
} }
else if( width > rBitmap.getWidth() ) else if( width > rBitmap.getWidth() )
{ {
msg_Dbg( getIntf(), "Bitmap width too small!" ); msg_Dbg( getIntf(), "bitmap width too small (%i)", rBitmap.getWidth() );
width = rBitmap.getWidth(); width = rBitmap.getWidth();
} }
if( height == -1 ) if( height == -1 )
...@@ -147,7 +147,8 @@ void X11Graphics::drawBitmap( const GenericBitmap &rBitmap, int xSrc, ...@@ -147,7 +147,8 @@ void X11Graphics::drawBitmap( const GenericBitmap &rBitmap, int xSrc,
} }
else if( height > rBitmap.getHeight() ) else if( height > rBitmap.getHeight() )
{ {
msg_Dbg( getIntf(), "Bitmap height too small!" ); msg_Dbg( getIntf(), "bitmap height too small (%i)", rBitmap.getHeight()
);
height = rBitmap.getHeight(); height = rBitmap.getHeight();
} }
...@@ -160,7 +161,7 @@ void X11Graphics::drawBitmap( const GenericBitmap &rBitmap, int xSrc, ...@@ -160,7 +161,7 @@ void X11Graphics::drawBitmap( const GenericBitmap &rBitmap, int xSrc,
// Safety check for debugging purpose // Safety check for debugging purpose
if( xDest + width > m_width || yDest + height > m_height ) if( xDest + width > m_width || yDest + height > m_height )
{ {
msg_Dbg( getIntf(), "Bitmap too large !" ); msg_Dbg( getIntf(), "bitmap too large" );
return; return;
} }
......
...@@ -175,7 +175,7 @@ void X11Loop::handleX11Event() ...@@ -175,7 +175,7 @@ void X11Loop::handleX11Event()
if( !pWin ) if( !pWin )
{ {
msg_Dbg( getIntf(), "No associated generic window !!" ); msg_Warn( getIntf(), "no associated generic window" );
return; return;
} }
...@@ -370,7 +370,7 @@ void X11Loop::handleX11Event() ...@@ -370,7 +370,7 @@ void X11Loop::handleX11Event()
((X11Factory*)pOsFactory)->m_dndMap[event.xany.window]; ((X11Factory*)pOsFactory)->m_dndMap[event.xany.window];
if( !pDnd ) if( !pDnd )
{ {
msg_Err( getIntf(), "No associated D&D object !!" ); msg_Err( getIntf(), "no associated D&D object" );
return; return;
} }
......
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