Minor fixes.

common/tool/tool_manager.cpp

@@ -316,6 +316,8 @@ bool TOOL_MANAGER::RunAction( const std::string& aActionName, bool aNow, void* a
         return true;
     }
 
+    wxASSERT_MSG( action != NULL, wxString::Format( _( "Could not find action %s." ), aActionName ) );
+
     return false;
 }
 

include/tool/tool_manager.h

@@ -116,7 +116,7 @@ public:
         return RunAction( aActionName, aNow, reinterpret_cast<void*>( aParam ) );
     }
 
-    bool RunAction( const std::string& aActionName, bool aNow, void* aParam = NULL );
+    bool RunAction( const std::string& aActionName, bool aNow, void* aParam );
 
     bool RunAction( const std::string& aActionName, bool aNow = false )
     {

include/view/view_item.h

@@ -203,10 +203,15 @@ public:
      */
     virtual void ViewUpdate( int aUpdateFlags = ALL )
     {
-        if( m_view && m_requiredUpdate == NONE )
-            m_view->MarkForUpdate( this );
+        if( m_view )
+        {
+            assert( aUpdateFlags != NONE );
+
+            if( m_requiredUpdate == NONE )
+                m_view->MarkForUpdate( this );
 
-        m_requiredUpdate |= aUpdateFlags;
+            m_requiredUpdate |= aUpdateFlags;
+        }
     }
 
     /**