Plot solder mask layer: fix incorrect margin of zones drawn on this layer (zone areas were slighly larger than the actual areas)

Fix a few  minor and very minor other issues: coverity warnings and compil warning.

common/class_marker_base.cpp

@@ -90,6 +90,7 @@ void MARKER_BASE::init()
 MARKER_BASE::MARKER_BASE( const MARKER_BASE& aMarker )
 {
     m_Pos = aMarker.m_Pos;
+    m_ErrorLevel = aMarker.m_ErrorLevel;
     m_MarkerType = aMarker.m_MarkerType;
     m_Color = aMarker.m_Color;
     m_ShapeBoundingBox = aMarker.m_ShapeBoundingBox;

include/geometry/shape_line_chain.h

@@ -254,8 +254,7 @@ public:
     {
         if( aIndex < 0 )
             aIndex += PointCount();
-
-        if( aIndex >= PointCount() )
+        else if( aIndex >= PointCount() )
             aIndex -= PointCount();
 
         return m_points[aIndex];

pcbnew/class_text_mod.cpp

@@ -1,9 +1,9 @@
 /*
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
- * Copyright (C) 2012 Jean-Pierre Charras, jean-pierre.charras@ujf-grenoble.fr
+ * Copyright (C) 2015 Jean-Pierre Charras, jp.charras at wanadoo.fr
  * Copyright (C) 2012 SoftPLC Corporation, Dick Hollenbeck <dick@softplc.com>
- * Copyright (C) 1992-2012 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2015 KiCad Developers, see AUTHORS.txt for contributors.
  *
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
@@ -108,7 +108,7 @@ void TEXTE_MODULE::Mirror( const wxPoint& aCentre, bool aMirrorAroundXAxis )
     // the mirror is around the Y axis or X axis if aMirrorAroundXAxis = true
     // the position is mirrored, but the text itself is not mirrored
     if( aMirrorAroundXAxis )
-        MIRROR( m_Pos.x, aCentre.x );
+        MIRROR( m_Pos.y, aCentre.y );
     else
         MIRROR( m_Pos.x, aCentre.x );
     NEGATE_AND_NORMALIZE_ANGLE_POS( m_Orient );

pcbnew/plot_board_layers.cpp

@@ -648,6 +648,8 @@ void PlotSolderMaskLayer( BOARD *aBoard, PLOTTER* aPlotter,
     itemplotter.SetLayerSet( aLayerMask );
 
     // Plot edge layer and graphic items
+    // They do not have a solder Mask margin, because they are only graphic items
+    // on this layer (like logos), not actually areas around pads.
     itemplotter.PlotBoardGraphicItems();
 
     for( MODULE* module = aBoard->m_Modules;  module;  module = module->Next() )
@@ -736,7 +738,13 @@ void PlotSolderMaskLayer( BOARD *aBoard, PLOTTER* aPlotter,
         }
     }
 
-    // Add filled zone areas
+    // Add filled zone areas.
+#if 0   // Set to 1 if a solder mask margin must be applied to zones on solder mask
+    int zone_margin = aBoard->GetDesignSettings().m_SolderMaskMargin;
+#else
+    int zone_margin = 0;
+#endif
+
     for( int ii = 0; ii < aBoard->GetAreaCount(); ii++ )
     {
         ZONE_CONTAINER* zone = aBoard->GetArea( ii );
@@ -745,9 +753,9 @@ void PlotSolderMaskLayer( BOARD *aBoard, PLOTTER* aPlotter,
             continue;
 
         zone->TransformOutlinesShapeWithClearanceToPolygon( areas,
-                    inflate, true );
+                    inflate+zone_margin, false );
         zone->TransformOutlinesShapeWithClearanceToPolygon( initialPolys,
-                    0, true );
+                    zone_margin, false );
     }
 
     // To avoid a lot of code, use a ZONE_CONTAINER

pcbnew/xchgmod.cpp

@@ -95,7 +95,8 @@ void PCB_EDIT_FRAME::InstallExchangeModuleFrame( MODULE* Module )
 {
     DIALOG_EXCHANGE_MODULE dialog( this, Module );
 
-    dialog.ShowQuasiModal();
+    int ret = dialog.ShowQuasiModal();
+    (void) ret;     // make coverity quiet.
 }
 
 

pcbnew/zones_convert_to_polygons_aux_functions.cpp

@@ -56,7 +56,7 @@ void ZONE_CONTAINER::TransformOutlinesShapeWithClearanceToPolygon(
     // add clearance to outline
     int clearance = aMinClearanceValue;
 
-    if( aUseNetClearance )
+    if( aUseNetClearance && IsOnCopperLayer() )
     {
         clearance = GetClearance();
         if( aMinClearanceValue > clearance )

potrace/greymap.cpp

@@ -1097,6 +1097,7 @@ int gm_print( FILE* f, greymap_t* gm )
                 }
             }
 
+            if( t )
                 fputc( "*#=- "[5 * d / t], f ); /* what a cute trick :) */
         }