kicad-developers team mailing list archive

Thread
Date

[PATCH] Need (path) in pretty library footprint?

To: KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx>
From: Dick Hollenbeck <dick@xxxxxxxxxxx>
Date: Fri, 04 Oct 2013 11:48:06 -0500
Openpgp: url=http://pool.sks-keyservers.net
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0

This patch keeps the (path) element out of pretty library (module)s.

OK?

Anything else we need like this?

=== modified file 'pcbnew/kicad_plugin.cpp'
--- pcbnew/kicad_plugin.cpp	2013-09-26 15:02:46 +0000
+++ pcbnew/kicad_plugin.cpp	2013-10-04 16:36:59 +0000
@@ -971,7 +971,7 @@
         m_out->Print( aNestLevel+1, "(tags %s)\n",
                       m_out->Quotew( aModule->GetKeywords() ).c_str() );
 
-    if( !aModule->GetPath().IsEmpty() )
+    if( !( m_ctl & CTL_OMIT_PATH ) && aModule->GetPath() )
         m_out->Print( aNestLevel+1, "(path %s)\n",
                       m_out->Quotew( aModule->GetPath() ).c_str() );
 

=== modified file 'pcbnew/kicad_plugin.h'
--- pcbnew/kicad_plugin.h	2013-09-26 05:29:54 +0000
+++ pcbnew/kicad_plugin.h	2013-10-04 16:39:02 +0000
@@ -42,6 +42,7 @@
 #define CTL_OMIT_NETS               (1 << 1)
 #define CTL_OMIT_TSTAMPS            (1 << 2)
 #define CTL_OMIT_INITIAL_COMMENTS   (1 << 3)    ///< omit MODULE initial comments
+#define CTL_OMIT_PATH               (1 << 4)
 
 // common combinations of the above:
 
@@ -49,7 +50,7 @@
 #define CTL_FOR_CLIPBOARD           (CTL_STD_LAYER_NAMES|CTL_OMIT_NETS)
 
 /// Format output for a footprint library instead of clipboard or BOARD
-#define CTL_FOR_LIBRARY             (CTL_STD_LAYER_NAMES|CTL_OMIT_NETS|CTL_OMIT_TSTAMPS)
+#define CTL_FOR_LIBRARY             (CTL_STD_LAYER_NAMES|CTL_OMIT_NETS|CTL_OMIT_TSTAMPS|CTL_OMIT_PATH)
 
 /// The zero arg constructor when PCB_IO is used for PLUGIN::Load() and PLUGIN::Save()ing
 /// a BOARD file underneath IO_MGR.