revert to using quotes around variable text which may be blank

pcbnew/gpcb_plugin.cpp

@@ -278,7 +278,7 @@ void GPCB_FPL_CACHE::Remove( const wxString& aFootprintName )
 
     if( it == m_modules.end() )
     {
-        THROW_IO_ERROR( wxString::Format( _( "library <%s> has no footprint %s to delete" ),
+        THROW_IO_ERROR( wxString::Format( _( "library <%s> has no footprint '%s' to delete" ),
                                           m_lib_path.GetPath().GetData(),
                                           aFootprintName.GetData() ) );
     }

pcbnew/kicad_plugin.cpp

@@ -249,7 +249,7 @@ void FP_CACHE::Remove( const wxString& aFootprintName )
 
     if( it == m_modules.end() )
     {
-        THROW_IO_ERROR( wxString::Format( _( "library <%s> has no footprint %s to delete" ),
+        THROW_IO_ERROR( wxString::Format( _( "library <%s> has no footprint '%s' to delete" ),
                                           m_lib_path.GetPath().GetData(),
                                           aFootprintName.GetData() ) );
     }
@@ -425,7 +425,7 @@ void PCB_IO::format( BOARD* aBoard, int aNestLevel ) const
     m_out->Print( aNestLevel, "(layers\n" );
 
     // Save only the used copper layers from front to back.
-    for( LAYER_NUM layer = LAST_COPPER_LAYER; layer >= FIRST_COPPER_LAYER; --layer) 
+    for( LAYER_NUM layer = LAST_COPPER_LAYER; layer >= FIRST_COPPER_LAYER; --layer)
     {
         LAYER_MSK mask = GetLayerMask( layer );
         if( mask & aBoard->GetEnabledLayers() )

pcbnew/legacy_plugin.cpp

@@ -4322,7 +4322,7 @@ void LEGACY_PLUGIN::FootprintDelete( const wxString& aLibraryPath, const wxStrin
     if( erasedCount != 1 )
     {
         THROW_IO_ERROR( wxString::Format(
-            _( "library <%s> has no footprint %s to delete" ),
+            _( "library <%s> has no footprint '%s' to delete" ),
             aLibraryPath.GetData(), aFootprintName.GetData() ) );
     }
 

pcbnew/specctra_export.cpp

@@ -549,7 +549,6 @@ IMAGE* SPECCTRA_DB::makeIMAGE( BOARD* aBoard, MODULE* aModule )
     TYPE_COLLECTOR  moduleItems;
     wxString        padName;
 
-
     // get all the MODULE's pads.
     moduleItems.Collect( aModule, scanPADs );
 
@@ -888,7 +887,7 @@ void SPECCTRA_DB::fillBOUNDARY( BOARD* aBoard, BOUNDARY* boundary ) throw( IO_ER
                 {
                     wxString error;
 
-                    error.Printf( _("Unsupported DRAWSEGMENT type %s"),
+                    error.Printf( _( "Unsupported DRAWSEGMENT type %s" ),
                         GetChars( BOARD_ITEM::ShowShape( (STROKE_T) graphic->GetShape() ) ) );
 
                     ThrowIOError( error );
@@ -964,7 +963,7 @@ void SPECCTRA_DB::FromBOARD( BOARD* aBoard ) throw( IO_ERROR )
 
             if( module->GetReference() == wxEmptyString )
             {
-                ThrowIOError( _("Component with value of %s has empty reference id."),
+                ThrowIOError( _( "Component with value of '%s' has empty reference id." ),
                                 GetChars( module->GetValue() ) );
             }
 
@@ -972,7 +971,7 @@ void SPECCTRA_DB::FromBOARD( BOARD* aBoard ) throw( IO_ERROR )
             STRINGSET_PAIR refpair = refs.insert( TO_UTF8( module->GetReference() ) );
             if( !refpair.second )      // insert failed
             {
-                ThrowIOError( _("Multiple components have identical reference IDs of %s."),
+                ThrowIOError( _( "Multiple components have identical reference IDs of '%s'." ),
                       GetChars( module->GetReference() ) );
             }
         }