Fix compatibility with some old V4 schematic files.

From master branch, commit #bfe59581

3d-viewer/3d_cache/sg/CMakeLists.txt

@@ -95,8 +95,12 @@ target_compile_definitions( kicad_3dsg PRIVATE -DCOMPILE_SGLIB )
 
 target_link_libraries( kicad_3dsg ${wxWidgets_LIBRARIES} )
 
+# Don't specify the ARCHIVE DESTINATION parameter to prevent
+# the install of the import library on Windows
+# https://cmake.org/pipermail/cmake/2011-November/047746.html
 install( TARGETS
     kicad_3dsg
-    DESTINATION ${KICAD_LIB}
+    RUNTIME DESTINATION ${KICAD_LIB}
+    LIBRARY DESTINATION ${KICAD_LIB}
     COMPONENT binary
     )

3d-viewer/3d_rendering/3d_render_raytracing/c3d_render_createscene.cpp

@@ -449,6 +449,10 @@ void C3D_RENDER_RAYTRACING::reload( REPORTER *aStatusTextReporter )
          ++ii )
     {
         PCB_LAYER_ID layer_id = static_cast<PCB_LAYER_ID>(ii->first);
+        const CBVHCONTAINER2D *container2d = static_cast<const CBVHCONTAINER2D *>(ii->second);
+
+        if( !container2d )
+            continue;
 
         // Mask kayers are not processed here because they are a special case
         if( (layer_id == B_Mask) || (layer_id == F_Mask) )
@@ -509,7 +513,6 @@ void C3D_RENDER_RAYTRACING::reload( REPORTER *aStatusTextReporter )
             break;
         }
 
-        const CBVHCONTAINER2D *container2d = static_cast<const CBVHCONTAINER2D *>(ii->second);
         const LIST_OBJECT2D &listObject2d = container2d->GetList();
 
         for( LIST_OBJECT2D::const_iterator itemOnLayer = listObject2d.begin();
@@ -655,6 +658,9 @@ void C3D_RENDER_RAYTRACING::reload( REPORTER *aStatusTextReporter )
             const CBVHCONTAINER2D *containerLayer2d =
                     static_cast<const CBVHCONTAINER2D *>(ii->second);
 
+            if( !containerLayer2d )
+                continue;
+
             // Only get the Solder mask layers
             if( !((layer_id == B_Mask) || (layer_id == F_Mask)) )
                 continue;

CMakeLists.txt

@@ -84,12 +84,12 @@ option( KICAD_SCRIPTING_ACTION_MENU
     ON )
 
 option( KICAD_USE_OCE
-    "Build tools and plugins related to OpenCascade Community Edition (default ON)"
-    ON )
+    "Build tools and plugins related to OpenCascade Community Edition (default OFF)"
+    OFF )
 
 option( KICAD_USE_OCC
-    "Build tools and plugins related to OpenCascade Technology (overrides KICAD_USE_OCE, default OFF)"
-    OFF )
+    "Build tools and plugins related to OpenCascade Technology (overrides KICAD_USE_OCE, default ON)"
+    ON )
 
 option( KICAD_INSTALL_DEMOS
     "Install KiCad demos and examples (default ON)"

CMakeModules/Findngspice.cmake

@@ -51,7 +51,6 @@ if( PKG_CONFIG_FOUND )
     endif()
 endif()
 
-
 find_path( NGSPICE_INCLUDE_DIR ngspice/sharedspice.h
     PATHS
         ${NGSPICE_ROOT_DIR}
@@ -65,7 +64,13 @@ find_path( NGSPICE_INCLUDE_DIR ngspice/sharedspice.h
         share/ngspice/include/ngspice
 )
 
-find_library( NGSPICE_LIBRARY ngspice
+if( UNIX )
+    set( NGSPICE_LIB_NAME libngspice.so.0 CACHE STRING "Optionally versioned name of the shared library" )
+else()
+    set( NGSPICE_LIB_NAME ngspice CACHE STRING "Optionally versioned name of the shared library" )
+endif()
+
+find_library( NGSPICE_LIBRARY ${NGSPICE_LIB_NAME}
     PATHS
         ${NGSPICE_ROOT_DIR}
         $ENV{NGSPICE_ROOT_DIR}
@@ -147,5 +152,5 @@ mark_as_advanced(
     NGSPICE_LIBRARY
     NGSPICE_DLL
     NGSPICE_BUILD_VERSION
-    NGSPIC_HAVE_CONFIG_H
+    NGSPICE_HAVE_CONFIG_H
 )

CMakeModules/KiCadVersion.cmake

@@ -2,7 +2,7 @@
 #  This program source code file is part of KICAD, a free EDA CAD application.
 #
 #  Copyright (C) 2016 Wayne Stambaugh <stambaughw@gmail.com>
-#  Copyright (C) 2016 - 2020 KiCad Developers, see AUTHORS.txt for contributors.
+#  Copyright (C) 2016 - 2021 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
@@ -35,4 +35,4 @@
 # be set after each version tag is added to the git repo.  This will
 # give developers a reasonable idea where which branch was used to build
 # KiCad.
-set( KICAD_VERSION "5.1.10" )
+set( KICAD_VERSION "5.1.12-unknown" )

Documentation/development/compiling.md

@@ -471,8 +471,8 @@ to download and [build Boost][] from source.  If you are building Boost on windo
 you will have to apply the Boost patches in the KiCad source [patches folder][].
 
 
-[download]: http://kicad-pcb.org/download/
-[KiCad website]: http://kicad-pcb.org/
+[download]: http://kicad.org/download/
+[KiCad website]: http://kicad.org/
 [KiCad Launchpad]: https://launchpad.net/kicad
 [GNU GCC]: https://gcc.gnu.org/
 [Clang]: http://clang.llvm.org/

Documentation/development/pcbnew-plugins.md

@@ -19,11 +19,11 @@ interpreter installed on the system.
 The Pcbnew Python API can be used stand-alone, i.e. no instance of Pcbnew is
 running and the board project to be manipulated is loaded and saved from and to
 file.  This approach is shown with some examples in the [user's
-documentation](http://docs.kicad-pcb.org/stable/en/pcbnew.html#_kicad_scripting_reference).
+documentation](http://docs.kicad.org/stable/en/pcbnew.html#_kicad_scripting_reference).
 
 Another documentation source is the auto-generated Doxygen reference of the
 API. It can be found
-[here](http://docs.kicad-pcb.org/doxygen-python/namespacepcbnew.html).
+[here](http://docs.kicad.org/doxygen-python/namespacepcbnew.html).
 
 # `Action Plugin` Support # {#ppi_action_pi}
 Besides the stand-alone usage of the generated Python plugin interface,

Documentation/development/road-map.md

@@ -101,5 +101,5 @@ capabilities.
 - In progress.
 
 
-[kicad-website]:http://kicad-pcb.org/
-[kicad-docs]:http://ci.kicad-pcb.org/job/kicad-doxygen/ws/Documentation/doxygen/html/index.html
+[kicad-website]:http://kicad.org/
+[kicad-docs]:http://ci.kicad.org/job/kicad-doxygen/ws/Documentation/doxygen/html/index.html

Documentation/development/stable-release-policy.md

@@ -71,7 +71,7 @@ GitHub plugin.
 
 To provide a convenient method for system packagers to build KiCad from known stable sources,
 source archives in the most common formats along with the resulting md5sum checksum will be
-added to either the KiCad developer's site on Launchpad or the main website at www.kicad-pcb.org.
+added to either the KiCad developer's site on Launchpad or the main website at www.kicad.org.
 
 # Stable Release Announcement # {#announcement}
 

Documentation/development/testing.md

@@ -304,6 +304,6 @@ Issues are still welcome for defects discovered.
 [Boost Unit Test framework]: https://www.boost.org/doc/libs/1_68_0/libs/test/doc/html/index.html
 [boost-test-functions]: https://www.boost.org/doc/libs/1_68_0/libs/test/doc/html/boost_test/utf_reference/testing_tool_ref.html
 [AFL fuzzing tool]: http://lcamtuf.coredump.cx/afl/
-[trace mask documentation]: http://docs.kicad-pcb.org/doxygen/group__trace__env__vars.html
-[trace mask documentation]: http://docs.kicad-pcb.org/doxygen/group__trace__env__vars.html
-[advanced config documentation]: http://docs.kicad-pcb.org/doxygen/namespaceAC__KEYS.html
+[trace mask documentation]: http://docs.kicad.org/doxygen/group__trace__env__vars.html
+[trace mask documentation]: http://docs.kicad.org/doxygen/group__trace__env__vars.html
+[advanced config documentation]: http://docs.kicad.org/doxygen/namespaceAC__KEYS.html

bitmap2component/CMakeLists.txt

@@ -37,7 +37,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE bitmap2component.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME bitmap2component )
 endif()
 

common/CMakeLists.txt

@@ -327,6 +327,7 @@ set( COMMON_SRCS
     kiway_express.cpp
     kiway_holder.cpp
     kiway_player.cpp
+    launch_ext.cpp
     lib_id.cpp
     lib_table_base.cpp
     lib_table_keywords.cpp

common/common.cpp

@@ -42,6 +42,7 @@
 #include <wx/url.h>
 
 #include <pgm_base.h>
+#include <clocale>
 
 using KIGFX::COLOR4D;
 
@@ -59,16 +60,33 @@ COLOR4D        g_GhostColor;
 
 // When reading/writing files, we need to swtich to setlocale( LC_NUMERIC, "C" ).
 // Works fine to read/write files with floating point numbers.
-// We can call setlocale( LC_NUMERIC, "C" ) of wxLocale( "C", "C", "C", false )
+// We can call setlocale( LC_NUMERIC, "C" ) or wxLocale( "C", "C", "C", false )
 // wxWidgets discourage a direct call to setlocale
 // However, for us, calling wxLocale( "C", "C", "C", false ) has a unwanted effect:
-// The I18N translations are no longer active, because the English dixtionary is selected.
+// The I18N translations are no longer active, because the English dictionary is selected.
 // To read files, this is not a major issues, but the resul can differ
 // from using setlocale(xx, "C").
-// Previouly, we called setlocale( LC_NUMERIC, "C" )
-// The old code will be removed when calling wxLocale( "C", "C", "C", false )
-// is fully tested, and all issues fixed
-#define USE_WXLOCALE 1      /* 0 to call setlocale, 1 to call wxLocale */
+// Previouly, we used only setlocale( LC_NUMERIC, "C" )
+//
+// Known issues are
+// on MSW
+//    using setlocale( LC_NUMERIC, "C" ) generates an alert message in debug mode,
+//    and this message ("Decimal separator mismatch") must be disabled.
+//    But calling wxLocale( "C", "C", "C", false ) works fine
+// On unix:
+//    calling wxLocale( "C", "C", "C", false ) breaks env vars containing non ASCII7 chars.
+//    these env vars return a empty string from wxGetEnv() in many cases, and if such a
+//    var must be read after calling wxLocale( "C", "C", "C", false ), it looks like empty
+//
+// So use wxLocale on Windows and setlocale on unix
+
+
+// set USE_WXLOCALE 0 to use setlocale, 1 to use wxLocale:
+#if defined( _WIN32 )
+#define USE_WXLOCALE 1
+#else
+#define USE_WXLOCALE 0
+#endif
 
 // On Windows, when using setlocale, a wx alert is generated
 // in some cases (reading a bitmap for instance)

common/dialog_about/AboutDialog_main.cpp

@@ -130,11 +130,7 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
     description << "<ul>";
     description << "<li>"
                 << _( "The official KiCad website - " )
-                << HtmlHyperlink( "http://www.kicad-pcb.org" )
-                << "</li>";
-    description << "<li>"
-                << _( "Developer website on Launchpad - " )
-                << HtmlHyperlink( "https://launchpad.net/kicad" )
+                << HtmlHyperlink( "https://www.kicad.org" )
                 << "</li>";
 
     description << "<li>"
@@ -163,7 +159,7 @@ static void buildKicadAboutBanner( EDA_BASE_FRAME* aParent, ABOUT_APP_INFO& aInf
     description << "<ul>";
     description << "<li>"
                 << _( "KiCad forum - " )
-                << HtmlHyperlink( "https://forum.kicad.info" )
+                << HtmlHyperlink( "https://go.kicad.org/forum" )
                 << "</li>";
 
     description << "</ul></p>";

common/eda_base_frame.cpp

@@ -47,7 +47,7 @@
 /// The default auto save interval is 10 minutes.
 #define DEFAULT_AUTO_SAVE_INTERVAL 600
 
-#define URL_GET_INVOLVED "http://kicad-pcb.org/contribute/"
+#define URL_GET_INVOLVED "https://www.kicad.org/contribute/"
 
 ///@{
 /// \ingroup config

common/launch_ext.cpp

@@ -0,0 +1,42 @@
+/*
+ * This program source code file is part of KiCad, a free EDA CAD application.
+ *
+ * Copyright (C) 2020 Jon Evans <jon@craftyjon.com>
+ * Copyright (C) 2020 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 as published by the
+ * Free Software Foundation, either version 3 of the License, or (at your
+ * option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <gestfich.h>
+#include <launch_ext.h>
+
+
+void LaunchExternal( const wxString& aPath )
+{
+#ifdef __WXMAC__
+    wxExecute( wxString::Format( "open \"%s\"", aPath ) );
+#else
+    wxString path( aPath );
+
+#if !wxCHECK_VERSION( 3, 1, 0 )
+    // Quote in case there are spaces in the path.
+    // Not needed on 3.1.4, but needed in 3.0 versions
+    // Moreover, on Linux, on 3.1.4 wx version, adding quotes breaks
+    // wxLaunchDefaultApplication
+    AddDelimiterString( path );
+#endif
+
+    wxLaunchDefaultApplication( path );
+#endif
+}

common/page_layout/page_layout_reader.cpp

@@ -128,6 +128,14 @@ void PAGE_LAYOUT_READER_PARSER::Parse( WORKSHEET_LAYOUT* aLayout )
         if( token == T_LEFT )
             token = NextTok();
 
+        if( token == T_kicad_wks || token == T_drawing_sheet )
+        {
+            THROW_PARSE_ERROR( _( "KiCad was unable to open this file because it was created with "
+                                  "a more recent version than the one you are running.\n\n"
+                                  "To open it you will need to upgrade KiCad to 5.99 or later." ),
+                               CurSource(), CurLine(), CurLineNumber(), CurOffset() );
+        }
+
         if( token == T_page_layout )
             continue;
 

common/page_layout/page_layout_reader.keywords

@@ -1,3 +1,5 @@
+drawing_sheet
+kicad_wks
 page_layout
 setup
 left_margin

cvpcb/cvpcb_mainframe.cpp

@@ -125,6 +125,7 @@ CVPCB_MAINFRAME::CVPCB_MAINFRAME( KIWAY* aKiway, wxWindow* aParent ) :
     m_libListBox            = NULL;
     m_mainToolBar           = NULL;
     m_modified              = false;
+    m_cannotClose           = false;
     m_skipComponentSelect   = false;
     m_filteringOptions      = 0;
     m_tcFilterString        = NULL;
@@ -307,6 +308,10 @@ void CVPCB_MAINFRAME::OnCloseWindow( wxCloseEvent& Event )
     // clear highlight symbol in schematic:
     SendMessageToEESCHEMA( true );
 
+
+    if( m_cannotClose )
+        return;
+
     // Delete window
     Destroy();
 }
@@ -1117,7 +1122,11 @@ void CVPCB_MAINFRAME::KiwayMailIn( KIWAY_EXPRESS& mail )
     switch( mail.Command() )
     {
     case MAIL_EESCHEMA_NETLIST:
+        // Disable Close events during ReadNetListAndFpFiles() to avoid crash when updating
+        // widgets:
+        m_cannotClose = true;
         ReadNetListAndFpFiles( payload );
+        m_cannotClose = false;
         /* @todo
         Go into SCH_EDIT_FRAME::OnOpenCvpcb( wxCommandEvent& event ) and trim GNL_ALL down.
         */

cvpcb/cvpcb_mainframe.h

@@ -80,11 +80,13 @@ public:
 
 protected:
     bool            m_modified;
-    bool            m_skipComponentSelect;      // true to skip OnSelectComponent event
-                                                // (in automatic selection/deletion of associations)
+    bool            m_skipComponentSelect;  // true to skip OnSelectComponent event
+                                            // (in automatic selection/deletion of associations)
     PARAM_CFG_ARRAY m_projectFileParams;
 
     bool            m_initialized;
+    bool            m_cannotClose;          // true when the cvpcb frame cannot be closed
+                                            // (mainly during reading a netlist sent by Eeschema)
 
     CVPCB_MAINFRAME( KIWAY* aKiway, wxWindow* aParent );
 

dxflib_qcad/dl_dxf.cpp

@@ -1714,7 +1714,7 @@ bool DL_Dxf::handleLWPolylineData( DL_CreationInterface*    /*creationInterface*
                 vertices[4 * vertexIndex + (groupCode / 10 - 1)] = toReal( groupValue );
             }
         }
-        else if( groupCode==42 && vertexIndex<maxVertices )
+        else if( groupCode==42 && vertexIndex<maxVertices && vertexIndex >= 0 )
         {
             vertices[4 * vertexIndex + 3] = toReal( groupValue );
         }

eeschema/CMakeLists.txt

@@ -9,8 +9,8 @@ if( KICAD_SPICE )
     set( INC_AFTER ${INC_AFTER} ${NGSPICE_INCLUDE_DIR} )
 
     # Find out the exact libngspice file name
-    get_filename_component( NGSPICE_DLL_REALPATH "${NGSPICE_DLL}" REALPATH )
-    get_filename_component( NGSPICE_DLL_FILE "${NGSPICE_DLL_REALPATH}" NAME )
+    get_filename_component( NGSPICE_DLL_ABSPATH "${NGSPICE_DLL}" ABSOLUTE )
+    get_filename_component( NGSPICE_DLL_FILE "${NGSPICE_DLL_ABSPATH}" NAME )
 
     set_property( SOURCE sim/ngspice.cpp
         APPEND PROPERTY COMPILE_DEFINITIONS
@@ -304,7 +304,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE eeschema.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME eeschema )
 endif()
 
@@ -423,11 +423,13 @@ if( APPLE )
         )
 
     if( KICAD_SPICE )
-        # bundle libngspice, codemodels and scripts
-        get_filename_component( REAL_LIBNGSPICE ${NGSPICE_LIBRARY} REALPATH )
-        get_filename_component( LIBNGSPICE_PATH ${REAL_LIBNGSPICE} DIRECTORY )
-        install( FILES "${REAL_LIBNGSPICE}"
-                 DESTINATION "${OSX_BUNDLE_INSTALL_PLUGIN_DIR}/sim" )
+        # bundle libngspice and codemodels
+        get_filename_component( ABS_LIBNGSPICE ${NGSPICE_LIBRARY} ABSOLUTE )
+        get_filename_component( LIBNGSPICE_PATH ${ABS_LIBNGSPICE} DIRECTORY )
+
+        install( DIRECTORY "${LIBNGSPICE_PATH}/"
+                DESTINATION "${OSX_BUNDLE_INSTALL_PLUGIN_DIR}/sim"
+                FILES_MATCHING PATTERN "*.dylib")
         install( DIRECTORY "${LIBNGSPICE_PATH}/ngspice"
                  DESTINATION "${OSX_BUNDLE_INSTALL_PLUGIN_DIR}/sim" )
         install( DIRECTORY "${LIBNGSPICE_PATH}/../share/ngspice/scripts"

eeschema/block.cpp

@@ -3,7 +3,7 @@
  *
  * Copyright (C) 2015 Jean-Pierre Charras, jp.charras at wanadoo.fr
  * Copyright (C) 2009 Wayne Stambaugh <stambaughw@gmail.com>
- * Copyright (C) 2004-2018 KiCad Developers, see change_log.txt for contributors.
+ * Copyright (C) 2004-2021 KiCad Developers, see change_log.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
@@ -451,7 +451,7 @@ void SCH_EDIT_FRAME::PasteListOfItems( wxDC* DC )
 {
     unsigned       i;
     SCH_ITEM*      item;
-    SCH_SHEET_LIST hierarchy( g_RootSheet );    // This is the entire schematic hierarcy.
+    SCH_SHEET_LIST hierarchy( g_RootSheet, false );    // This is the entire schematic hierarcy.
 
     if( m_blockItems.GetCount() == 0 )
     {
@@ -468,8 +468,9 @@ void SCH_EDIT_FRAME::PasteListOfItems( wxDC* DC )
     // the destination sheet. Moreover new sheets create new sheetpaths, and component
     // alternante references must be created and cleared
     bool hasSheetPasted = false;
+
     // Keep trace of existing sheet paths. Paste block can modify this list
-    SCH_SHEET_LIST initial_sheetpathList( g_RootSheet );
+    SCH_SHEET_LIST initial_sheetpathList( g_RootSheet, false );
 
     for( i = 0; i < m_blockItems.GetCount(); i++ )
     {

eeschema/bom_plugins.cpp

@@ -2,6 +2,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2018 CERN
+ * Copyright (C) 2021 KiCad Developers, see AUTHORS.txt for contributors.
  * @author Maciej Suminski <maciej.suminski@cern.ch>
  *
  * This program is free software; you can redistribute it and/or

eeschema/dialogs/dialog_bom.cpp

@@ -457,7 +457,6 @@ void DIALOG_BOM::OnRunPlugin( wxCommandEvent& event )
     // Calculate the xml netlist filename
     wxFileName fn = g_RootSheet->GetScreen()->GetFileName();
 
-    fn.SetPath( wxPathOnly( Prj().GetProjectFullName() ) );
     fn.ClearExt();
 
     wxString fullfilename = fn.GetFullPath();

eeschema/dialogs/dialog_bom_help.html

@@ -44,7 +44,7 @@ Full documentation:</h1>
 <font face="Times New Roman, serif"><font size="3" style="font-size: 12pt"><span style="font-weight: normal">The
 </span><i><b>Eeschema documentation</b></i> <span style="font-weight: normal">describes
 this </span><b>intermediate netlist and gives examples<br></b><span style="font-weight: normal">See </span>
-<i><b>http://docs.kicad-pcb.org/stable/en/eeschema.html#creating-customized-netlists-and-bom-files</b></i></font></font></p>
+<i><b>https://go.kicad.org/docs/5.1/en/eeschema/eeschema.html#creating-customized-netlists-and-bom-files</b></i></font></font></p>
 <h1 class="western"><i>2 - </i>The intermediate Netlist File</h1>
 <p lang="en-US" class="western" style="margin-bottom: 0cm; widows: 0; orphans: 0; page-break-before: auto; page-break-after: auto">
 <font face="Times New Roman, serif"><font size="3" style="font-size: 12pt">BOM

eeschema/dialogs/dialog_spice_model.cpp

@@ -193,7 +193,7 @@ bool DIALOG_SPICE_MODEL::TransferDataFromWindow()
 
         int modelIdx = m_modelType->GetSelection();
 
-        if( modelIdx > 0 && modelIdx < (int)modelTypes.size() )
+        if( modelIdx >= 0 && modelIdx < (int)modelTypes.size() )
             m_fieldsTmp[SF_PRIMITIVE] = static_cast<char>( modelTypes[modelIdx].type );
 
         m_fieldsTmp[SF_MODEL] = m_modelName->GetValue();

eeschema/find.cpp

@@ -3,7 +3,7 @@
  *
  * Copyright (C) 2004 Jean-Pierre Charras, jaen-pierre.charras@gipsa-lab.inpg.com
  * Copyright (C) 2008 Wayne Stambaugh <stambaughw@gmail.com>
- * Copyright (C) 2004-2018 KiCad Developers, see change_log.txt for contributors.
+ * Copyright (C) 2004-2021 KiCad Developers, see change_log.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
@@ -124,7 +124,7 @@ SCH_ITEM* SCH_EDIT_FRAME::FindComponentAndItem( const wxString& aReference,
     if( !aSearchHierarchy )
         sheetList.push_back( *m_CurrentSheet );
     else
-        sheetList.BuildSheetList( g_RootSheet );
+        sheetList.BuildSheetList( g_RootSheet, false );
 
     for( SCH_SHEET_PATHS_ITER it = sheetList.begin(); it != sheetList.end(); ++it )
     {

eeschema/netlist_exporters/netlist_exporter_generic.cpp

@@ -31,6 +31,7 @@
 #include <class_library.h>
 #include <sch_base_frame.h>
 #include <symbol_lib_table.h>
+#include <wx/wfstream.h>
 
 
 static bool sortPinsByNumber( LIB_PIN* aPin1, LIB_PIN* aPin2 );
@@ -41,12 +42,19 @@ bool NETLIST_EXPORTER_GENERIC::WriteNetlist( const wxString& aOutFileName, unsig
     for( unsigned ii = 0; ii < m_masterList->size(); ii++ )
         m_masterList->GetItem( ii )->m_Flag = 0;
 
+    // declare the stream ourselves to use the buffered FILE api
+    // instead of letting wx use the syscall variant
+    wxFFileOutputStream stream( aOutFileName );
+
+    if( !stream.IsOk() )
+        return false;
+
     // output the XML format netlist.
     wxXmlDocument   xdoc;
 
     xdoc.SetRoot( makeRoot( GNL_ALL ) );
 
-    return xdoc.Save( aOutFileName, 2 /* indent bug, today was ignored by wxXml lib */ );
+    return xdoc.Save( stream, 2 /* indent bug, today was ignored by wxXml lib */ );
 }
 
 

eeschema/plot_schematic_DXF.cpp

@@ -5,7 +5,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 1992-2010 Lorenzo Marcantonio
- * Copyright (C) 1992-2017 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -50,7 +50,7 @@ void DIALOG_PLOT_SCHEMATIC::CreateDXFFile( bool aPlotAll, bool aPlotFrameRef )
     SCH_SHEET_LIST sheetList;
 
     if( aPlotAll )
-        sheetList.BuildSheetList( g_RootSheet );
+        sheetList.BuildSheetList( g_RootSheet, false );
     else
         sheetList.push_back( schframe->GetCurrentSheet() );
 

eeschema/plot_schematic_HPGL.cpp

@@ -5,7 +5,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 1992-2010 Jean-Pierre Charras jp.charras at wanadoo.fr
- * Copyright (C) 1992-2017 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -96,7 +96,7 @@ void DIALOG_PLOT_SCHEMATIC::createHPGLFile( bool aPlotAll, bool aPlotFrameRef )
     SCH_SHEET_LIST  sheetList;
 
     if( aPlotAll )
-        sheetList.BuildSheetList( g_RootSheet );
+        sheetList.BuildSheetList( g_RootSheet, false );
     else
         sheetList.push_back( m_parent->GetCurrentSheet() );
 

eeschema/plot_schematic_PDF.cpp

@@ -5,7 +5,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 1992-2010 Jean-Pierre Charras jp.charras at wanadoo.fr
- * Copyright (C) 1992-2017 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -53,7 +53,7 @@ void DIALOG_PLOT_SCHEMATIC::createPDFFile( bool aPlotAll, bool aPlotFrameRef )
     SCH_SHEET_LIST sheetList;
 
     if( aPlotAll )
-        sheetList.BuildSheetList( g_RootSheet );
+        sheetList.BuildSheetList( g_RootSheet, false );
     else
         sheetList.push_back( m_parent->GetCurrentSheet() );
 

eeschema/plot_schematic_PS.cpp

@@ -4,7 +4,7 @@
 /*
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
- * Copyright (C) 1992-2016 KiCad Developers, see change_log.txt for contributors.
+ * Copyright (C) 1992-2021 KiCad Developers, see change_log.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
@@ -51,7 +51,7 @@ void DIALOG_PLOT_SCHEMATIC::createPSFile( bool aPlotAll, bool aPlotFrameRef )
     SCH_SHEET_LIST  sheetList;
 
     if( aPlotAll )
-        sheetList.BuildSheetList( g_RootSheet );
+        sheetList.BuildSheetList( g_RootSheet, false );
     else
         sheetList.push_back( m_parent->GetCurrentSheet() );
 

eeschema/plot_schematic_SVG.cpp

@@ -2,8 +2,8 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2009 Jean-Pierre Charras, jp.charras at wanadoo.fr
- * Copyright (C) 2011-2016 Wayne Stambaugh <stambaughw@verizon.net>
- * Copyright (C) 1992-2016 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 2011 Wayne Stambaugh <stambaughw@gmail.com>
+ * Copyright (C) 1992-2021 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
@@ -48,7 +48,7 @@ void DIALOG_PLOT_SCHEMATIC::createSVGFile( bool aPrintAll, bool aPrintFrameRef )
     SCH_SHEET_LIST  sheetList;
 
     if( aPrintAll )
-        sheetList.BuildSheetList( g_RootSheet );
+        sheetList.BuildSheetList( g_RootSheet, false );
     else
         sheetList.push_back( m_parent->GetCurrentSheet() );
 

eeschema/sch_component.cpp

@@ -922,6 +922,11 @@ wxString SCH_COMPONENT::GetFieldText( const wxString& aFieldName, SCH_EDIT_FRAME
     {
         if( aFieldName == m_Fields[ii].GetName() )
             return m_Fields[ii].GetText();
+
+        if( ii < MANDATORY_FIELDS && aFieldName == TEMPLATE_FIELDNAME::GetCanonicalFieldName( ii ) )
+        {
+            return m_Fields[ii].GetText();
+        }
     }
 
     return wxEmptyString;

eeschema/sch_edit_frame.cpp

@@ -2,7 +2,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2017 Jean-Pierre Charras, jp.charras at wanadoo.fr
- * Copyright (C) 1992-2018 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -507,7 +507,7 @@ void SCH_EDIT_FRAME::SetSheetNumberAndCount()
     int            sheet_count = g_RootSheet->CountSheets();
     int            SheetNumber = 1;
     wxString       current_sheetpath = m_CurrentSheet->Path();
-    SCH_SHEET_LIST sheetList( g_RootSheet );
+    SCH_SHEET_LIST sheetList( g_RootSheet, false );
 
     // Examine all sheets path to find the current sheets path,
     // and count them from root to the current sheet path:
@@ -642,7 +642,8 @@ void SCH_EDIT_FRAME::OnCloseWindow( wxCloseEvent& aEvent )
 {
     if( Kiface().IsSingle() )
     {
-        LIB_EDIT_FRAME* libeditFrame = (LIB_EDIT_FRAME*) Kiway().Player( FRAME_SCH_LIB_EDITOR, false );
+        LIB_EDIT_FRAME* libeditFrame = (LIB_EDIT_FRAME*) Kiway().Player( FRAME_SCH_LIB_EDITOR,
+                                                                         false );
         if( libeditFrame && !libeditFrame->Close() )   // Can close component editor?
             return;
 
@@ -661,7 +662,7 @@ void SCH_EDIT_FRAME::OnCloseWindow( wxCloseEvent& aEvent )
     if( simFrame && !simFrame->Close() )   // Can close the simulator?
         return;
 
-    SCH_SHEET_LIST sheetList( g_RootSheet );
+    SCH_SHEET_LIST sheetList( g_RootSheet, false );
 
     if( sheetList.IsModified() )
     {
@@ -1377,7 +1378,6 @@ void SCH_EDIT_FRAME::addCurrentItemToScreen()
 
         undoItem = parentSheet;
     }
-
     else if( item->Type() == SCH_FIELD_T )
     {
         parentComponent = (SCH_COMPONENT*) item->GetParent();
@@ -1394,7 +1394,7 @@ void SCH_EDIT_FRAME::addCurrentItemToScreen()
         // for all new sheet paths added by the new sheet (if this sheet is loaded from
         // and existing sheet or a existing file, it can also contain subsheets)
         bool doClearAnnotation = false;
-        SCH_SHEET_LIST initial_sheetpathList( g_RootSheet );
+        SCH_SHEET_LIST initial_sheetpathList( g_RootSheet, false );
 
         if( item->Type() == SCH_SHEET_T )
         {
@@ -1506,7 +1506,9 @@ void SCH_EDIT_FRAME::UpdateTitle()
                 title += _( " [Read Only]" );
         }
         else
+        {
             title += _( " [no file]" );
+        }
     }
 
     SetTitle( title );

eeschema/sch_legacy_plugin.cpp

@@ -2,7 +2,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2016 CERN
- * Copyright (C) 2016-2019 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 2016-2021 KiCad Developers, see AUTHORS.txt for contributors.
  *
  * @author Wayne Stambaugh <stambaughw@gmail.com>
  *
@@ -1112,9 +1112,13 @@ SCH_BITMAP* SCH_LEGACY_PLUGIN::loadBitmap( FILE_LINE_READER& aReader )
                 // Read PNG data, stored in hexadecimal,
                 // each byte = 2 hexadecimal digits and a space between 2 bytes
                 // and put it in memory stream buffer
+                // Note:
+                // Some old files created by the V4 schematic versions have a extra
+                // "$EndBitmap" at the end of the hexadecimal data. (Probably due to
+                // a bug). So discard it
                 int len = strlen( line );
 
-                for( ; len > 0 && !isspace( *line ); len -= 3, line += 3 )
+                for( ; len > 0 && !isspace( *line ) && '$' != *line; len -= 3, line += 3 )
                 {
                     int value = 0;
 

eeschema/sch_sheet_path.cpp

@@ -3,7 +3,7 @@
  *
  * Copyright (C) 2017 Jean-Pierre Charras, jp.charras at wanadoo.fr
  * Copyright (C) 2011 Wayne Stambaugh <stambaughw@gmail.com>
- * Copyright (C) 1992-2017 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -180,7 +180,6 @@ void SCH_SHEET_PATH::UpdateAllScreenReferences()
 }
 
 
-
 void SCH_SHEET_PATH::GetComponents( SCH_REFERENCE_LIST& aReferences, bool aIncludePowerSymbols,
                                     bool aForceIncludeOrphanComponents )
 {
@@ -423,15 +422,12 @@ SCH_SHEET* SCH_SHEET_PATH::FindSheetByName( const wxString& aSheetName )
 }
 
 
-/********************************************************************/
-/* Class SCH_SHEET_LIST to handle the list of Sheets in a hierarchy */
-/********************************************************************/
-SCH_SHEET_LIST::SCH_SHEET_LIST( SCH_SHEET* aSheet )
+SCH_SHEET_LIST::SCH_SHEET_LIST( SCH_SHEET* aSheet, bool aCheckIntegrity )
 {
     m_isRootSheet = false;
 
     if( aSheet != NULL )
-        BuildSheetList( aSheet );
+        BuildSheetList( aSheet, aCheckIntegrity );
 }
 
 
@@ -451,7 +447,7 @@ SCH_SHEET_PATH* SCH_SHEET_LIST::GetSheetByPath( const wxString& aPath, bool aHum
 }
 
 
-void SCH_SHEET_LIST::BuildSheetList( SCH_SHEET* aSheet )
+void SCH_SHEET_LIST::BuildSheetList( SCH_SHEET* aSheet, bool aCheckIntegrity )
 {
     wxCHECK_RET( aSheet != NULL, wxT( "Cannot build sheet list from undefined sheet." ) );
 
@@ -480,21 +476,30 @@ void SCH_SHEET_LIST::BuildSheetList( SCH_SHEET* aSheet )
             {
                 SCH_SHEET* sheet = (SCH_SHEET*) item;
 
-                if( !m_currentSheetPath.TestForRecursion(
-                            sheet->GetFileName(), aSheet->GetFileName() ) )
-                    BuildSheetList( sheet );
+                if( aCheckIntegrity )
+                {
+                    if( !m_currentSheetPath.TestForRecursion( sheet->GetFileName(),
+                                                              aSheet->GetFileName() ) )
+                        BuildSheetList( sheet, true );
+                    else
+                        badSheets.push_back( sheet );
+                }
                 else
-                    badSheets.push_back( sheet );
+                {
+                    BuildSheetList( sheet, false );
+                }
             }
 
             item = item->Next();
         }
 
-
-        for( auto sheet : badSheets )
+        if( aCheckIntegrity )
         {
-            aSheet->GetScreen()->Remove( sheet );
-            aSheet->GetScreen()->SetModify();
+            for( auto sheet : badSheets )
+            {
+                aSheet->GetScreen()->Remove( sheet );
+                aSheet->GetScreen()->SetModify();
+            }
         }
     }
 
@@ -609,6 +614,7 @@ void SCH_SHEET_LIST::GetComponents( SCH_REFERENCE_LIST& aReferences, bool aInclu
         (*it).GetComponents( aReferences, aIncludePowerSymbols, aForceIncludeOrphanComponents );
 }
 
+
 void SCH_SHEET_LIST::GetMultiUnitComponents( SCH_MULTI_UNIT_REFERENCE_MAP &aRefList,
                                              bool aIncludePowerSymbols )
 {

eeschema/sch_sheet_path.h

@@ -3,7 +3,7 @@
  *
  * Copyright (C) 2017 Jean-Pierre Charras, jp.charras at wanadoo.fr
  * Copyright (C) 2011 Wayne Stambaugh <stambaughw@gmail.com>
- * Copyright (C) 1992-2017 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -330,12 +330,11 @@ private:
 public:
 
     /**
-     * Constructor
-     * build a flattened list of SCH_SHEET_PATH objects from \a aSheet.
+     * Build a flattened list of SCH_SHEET_PATH objects from \a aSheet.
      *
-     * If aSheet == NULL, then this is an empty hierarchy which the user can populate.
+     * If aSheet == NULL, then this is the full hierarchy built from the root sheet.
      */
-    SCH_SHEET_LIST( SCH_SHEET* aSheet = NULL );
+    SCH_SHEET_LIST( SCH_SHEET* aSheet = nullptr, bool aCheckIntegrity = true );
 
     ~SCH_SHEET_LIST() {}
 
@@ -470,15 +469,17 @@ public:
     SCH_SHEET* FindSheetByName( const wxString& aSheetName );
 
     /**
-     * Function BuildSheetList
-     * builds the list of sheets and their sheet path from \a aSheet.
+     * Build the list of sheets and their sheet path from \a aSheet.
+     *
      * If \a aSheet is the root sheet, the full sheet path and sheet list are built.
      *
      * @param aSheet is the starting sheet from which the list is built, or NULL
      *               indicating that g_RootSheet should be used.
+     * @param aIntegrityCheck set to true to test for sheet recursion while building the
+     *                        hierarchy.
      * @throw std::bad_alloc if the memory for the sheet path list could not be allocated.
      */
-    void BuildSheetList( SCH_SHEET* aSheet );
+    void BuildSheetList( SCH_SHEET* aSheet, bool aCheckIntegrity );
 };
 
 #endif // CLASS_DRAWSHEET_PATH_H

eeschema/sim/sim_plot_panel.cpp

@@ -2,6 +2,8 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2016 CERN
+ * Copyright (C) 2021 KiCad Developers, see AUTHORS.txt for contributors.
+ *
  * @author Tomasz Wlostowski <tomasz.wlostowski@cern.ch>
  * @author Maciej Suminski <maciej.suminski@cern.ch>
  *
@@ -522,7 +524,11 @@ bool SIM_PLOT_PANEL::AddTrace( const wxString& aName, int aPoints,
         else
         {
             for( int i = 0; i < aPoints; i++ )
-                tmp[i] = 20 * log( tmp[i] ) / log( 10.0 );      // convert to dB
+            {
+                // log( 0 ) is not valid.
+                if( tmp[i] != 0.0 )
+                    tmp[i] = 20 * log( tmp[i] ) / log( 10.0 );  // convert to dB
+            }
         }
     }
 

eeschema/template_fieldnames.cpp

@@ -31,6 +31,36 @@
 using namespace TFIELD_T;
 
 
+const wxString TEMPLATE_FIELDNAME::GetCanonicalFieldName( int aFieldNdx )
+{
+    // Fixed values for the first few default fields used by EESCHEMA
+    // (mandatory fields)
+    switch( aFieldNdx )
+    {
+    case  REFERENCE:
+        return wxT( "Reference" );   // The component reference, R1, C1, etc.
+
+    case  VALUE:
+        return wxT( "Value" );       // The component value + name
+
+    case  FOOTPRINT:
+        return wxT( "Footprint" );     // The footprint for use with Pcbnew
+
+    case  DATASHEET:
+        return wxT( "Datasheet" );   // Link to a datasheet for component
+
+    default:
+        break;
+    }
+
+    // Other fields are use fields, give a default name:
+    wxString fieldName = wxT( "Field" );
+    fieldName << aFieldNdx;
+    return fieldName;
+}
+
+
+
 const wxString TEMPLATE_FIELDNAME::GetDefaultFieldName( int aFieldNdx )
 {
     static void* locale = nullptr;
@@ -52,6 +82,8 @@ const wxString TEMPLATE_FIELDNAME::GetDefaultFieldName( int aFieldNdx )
         locale = Pgm().GetLocale();
     }
 
+    // Fixed values for the first few default fields used by EESCHEMA
+
     // Fixed values for the first few default fields used by EESCHEMA
     // (mandatory fields)
     switch( aFieldNdx )
@@ -78,6 +110,7 @@ const wxString TEMPLATE_FIELDNAME::GetDefaultFieldName( int aFieldNdx )
     return fieldName;
 }
 
+
 void TEMPLATE_FIELDNAME::Format( OUTPUTFORMATTER* out, int nestLevel ) const
 {
     out->Print( nestLevel, "(field (name %s)",  out->Quotew( m_Name ).c_str() );

eeschema/template_fieldnames.h

@@ -122,6 +122,14 @@ struct TEMPLATE_FIELDNAME
      * @param aFieldNdx The field number index, > 0
      */
     static const wxString GetDefaultFieldName( int aFieldNdx );
+
+    /**
+     * returns a not translated default symbol field name for field \a aFieldNdx
+     * for all components.
+     * These fieldnames are not modifiable, but template fieldnames are.
+     * @param aFieldNdx The field number index, > 0
+     */
+    static const wxString GetCanonicalFieldName( int aFieldNdx );
 };
 
 typedef std::vector< TEMPLATE_FIELDNAME > TEMPLATE_FIELDNAMES;

gerbview/CMakeLists.txt

@@ -102,7 +102,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE gerbview.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME gerbview )
 endif()
 

gerbview/excellon_image.h

@@ -155,11 +155,10 @@ public: EXCELLON_IMAGE( int layer ) :
 
     ~EXCELLON_IMAGE() {};
 
-    virtual void ResetDefaultValues() override
-    {
-        GERBER_FILE_IMAGE::ResetDefaultValues();
-        SelectUnits( false );
-    }
+    /**
+     * Set all parameters to a default value, before reading a file
+     */
+    virtual void ResetDefaultValues() override;
 
 
     /**

gerbview/excellon_read_drill_file.cpp

@@ -2,7 +2,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 1992-2016 Jean-Pierre Charras <jp.charras at wanadoo.fr>
- * Copyright (C) 1992-2018 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -300,6 +300,19 @@ bool GERBVIEW_FRAME::Read_EXCELLON_File( const wxString& aFullFileName )
     return success;
 }
 
+
+void EXCELLON_IMAGE::ResetDefaultValues()
+{
+    GERBER_FILE_IMAGE::ResetDefaultValues();
+    SelectUnits( false );       // Default unit = inch
+
+    // Files using non decimal can use No Trailing zeros or No leading Zeros
+    // Unfortunately, the identifier (INCH,TZ or INCH,LZ for instance) is not
+    // always set in drill files.
+    // The option leading zeros looks like more frequent, so use this default
+    m_NoTrailingZeros = true;
+}
+
 /*
  * Read a EXCELLON file.
  * Gerber classes are used because there is likeness between Gerber files
@@ -312,7 +325,6 @@ bool GERBVIEW_FRAME::Read_EXCELLON_File( const wxString& aFullFileName )
  *   integer 2.4 format in imperial units,
  *   integer 3.2 or 3.3 format (metric units).
  */
-
 bool EXCELLON_IMAGE::LoadFile( const wxString & aFullFileName )
 {
     // Set the default parmeter values:
@@ -373,7 +385,8 @@ bool EXCELLON_IMAGE::LoadFile( const wxString & aFullFileName )
                 }
                 break;
 
-            case 'T': // Tool command
+            case 'T':               // Select Tool command (can also create
+                                    // the tool with an embedded definition)
                 Select_Tool( text );
                 break;
 
@@ -479,8 +492,7 @@ bool EXCELLON_IMAGE::Execute_HEADER_And_M_Command( char*& text )
         if( *text != ',' )
         {
             // No TZ or LZ specified. Should be a decimal format
-            // but this is not always the case. Use default TZ setting as default
-            m_NoTrailingZeros = false;
+            // but this is not always the case. Use our default setting
             break;
         }
 
@@ -724,15 +736,20 @@ bool EXCELLON_IMAGE::Select_Tool( char*& text )
             dcode_id = TOOLS_MAX_COUNT - 1;
 
         m_Current_Tool = dcode_id;
-        D_CODE* currDcode = GetDCODEOrCreate( dcode_id, true );
+        D_CODE* currDcode = GetDCODE( dcode_id );
 
-        if( currDcode == NULL && tool_id > 0 )   // if the definition is embedded, enter it
+        // if nopt existing, and the definition is embedded, create it
+        if( currDcode == NULL && tool_id > 0 )
         {
             text = startline;   // text starts at the beginning of the command
             readToolInformation( text );
             currDcode = GetDCODE( dcode_id );
         }
 
+        // If the Tool is really not existing, create a dummy tool
+        if( !currDcode )
+            currDcode = GetDCODEOrCreate( dcode_id, true );
+
         if( currDcode )
             currDcode->m_InUse = true;
     }

gerbview/gerber_draw_item.cpp

@@ -371,10 +371,21 @@ const EDA_RECT GERBER_DRAW_ITEM::GetBoundingBox() const
     {
         if( code && code->m_Shape == APT_RECT )
         {
-            if( m_Polygon.OutlineCount() > 0 )
+            if( m_Polygon.OutlineCount() == 0 )
             {
-                auto bb = m_Polygon.BBox();
-                bbox.Inflate( bb.GetWidth() / 2, bb.GetHeight() / 2 );
+                // We cannot initialize m_Polygon, because we are in a const function.
+                // So use a temporary polygon
+                SHAPE_POLY_SET poly_shape;
+                ConvertSegmentToPolygon( &poly_shape );
+                BOX2I bb = poly_shape.BBox();
+                bbox.SetSize( bb.GetWidth(), bb.GetHeight() );
+                bbox.SetOrigin( bb.GetOrigin().x, bb.GetOrigin().y );
+            }
+
+            else
+            {
+                BOX2I bb = m_Polygon.BBox();
+                bbox.SetSize( bb.GetWidth(), bb.GetHeight() );
                 bbox.SetOrigin( bb.GetOrigin().x, bb.GetOrigin().y );
             }
         }
@@ -593,10 +604,10 @@ void GERBER_DRAW_ITEM::Draw( EDA_DRAW_PANEL* aPanel, wxDC* aDC, GR_DRAWMODE aDra
 }
 
 
-void GERBER_DRAW_ITEM::ConvertSegmentToPolygon()
+void GERBER_DRAW_ITEM::ConvertSegmentToPolygon( SHAPE_POLY_SET* aPolygon ) const
 {
-    m_Polygon.RemoveAllContours();
-    m_Polygon.NewOutline();
+    aPolygon->RemoveAllContours();
+    aPolygon->NewOutline();
 
     wxPoint start = m_Start;
     wxPoint end = m_End;
@@ -626,31 +637,31 @@ void GERBER_DRAW_ITEM::ConvertSegmentToPolygon()
     corner.x -= m_Size.x/2;
     corner.y -= m_Size.y/2;
     wxPoint close = corner;
-    m_Polygon.Append( VECTOR2I( corner ) );  // Lower left corner, start point (1)
+    aPolygon->Append( VECTOR2I( corner ) );  // Lower left corner, start point (1)
     corner.y += m_Size.y;
-    m_Polygon.Append( VECTOR2I( corner ) );  // upper left corner, start point (2)
+    aPolygon->Append( VECTOR2I( corner ) );  // upper left corner, start point (2)
 
     if( delta.x || delta.y)
     {
         corner += delta;
-        m_Polygon.Append( VECTOR2I( corner ) );  // upper left corner, end point (3)
+        aPolygon->Append( VECTOR2I( corner ) );  // upper left corner, end point (3)
     }
 
     corner.x += m_Size.x;
-    m_Polygon.Append( VECTOR2I( corner ) );  // upper right corner, end point (4)
+    aPolygon->Append( VECTOR2I( corner ) );  // upper right corner, end point (4)
     corner.y -= m_Size.y;
-    m_Polygon.Append( VECTOR2I( corner ) );  // lower right corner, end point (5)
+    aPolygon->Append( VECTOR2I( corner ) );  // lower right corner, end point (5)
 
     if( delta.x || delta.y )
     {
         corner -= delta;
-        m_Polygon.Append( VECTOR2I( corner ) );  // lower left corner, start point (6)
+        aPolygon->Append( VECTOR2I( corner ) );  // lower left corner, start point (6)
     }
 
-    m_Polygon.Append( VECTOR2I( close ) );  // close the shape
+    aPolygon->Append( VECTOR2I( close ) );  // close the shape
 
     // Create final polygon:
-    for( auto it = m_Polygon.Iterate( 0 ); it; ++it )
+    for( auto it = aPolygon->Iterate( 0 ); it; ++it )
     {
         if( change )
             ( *it ).y = -( *it ).y;
@@ -660,6 +671,12 @@ void GERBER_DRAW_ITEM::ConvertSegmentToPolygon()
 }
 
 
+void GERBER_DRAW_ITEM::ConvertSegmentToPolygon()
+{
+    ConvertSegmentToPolygon( &m_Polygon );
+}
+
+
 void GERBER_DRAW_ITEM::DrawGbrPoly( EDA_RECT*      aClipBox,
                                     wxDC*          aDC,
                                     COLOR4D        aColor,

gerbview/gerber_draw_item.h

@@ -245,8 +245,11 @@ public:
      * convert a line to an equivalent polygon.
      * Useful when a line is plotted using a rectangular pen.
      * In this case, the usual segment plot function cannot be used
-     */
-    void ConvertSegmentToPolygon();
+     * @param aPolygon is the SHAPE_POLY_SET to fill. If null (usual case),
+     * m_Polygon will be used
+      */
+     void ConvertSegmentToPolygon();
+    void ConvertSegmentToPolygon( SHAPE_POLY_SET* aPolygon ) const;
 
     /**
      * Function DrawGbrPoly

gerbview/gerber_file_image.h

@@ -267,6 +267,9 @@ public:
      */
     int GetDcodesCount();
 
+    /**
+     * Set all parameters to a default value, before reading a file
+     */
     virtual void ResetDefaultValues();
 
     COLOR4D GetPositiveDrawColor() const { return m_PositiveDrawColor; }

include/launch_ext.h

@@ -0,0 +1,32 @@
+/*
+ * This program source code file is part of KiCad, a free EDA CAD application.
+ *
+ * Copyright (C) 2020 Jon Evans <jon@craftyjon.com>
+ * Copyright (C) 2020 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 as published by the
+ * Free Software Foundation, either version 3 of the License, or (at your
+ * option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#ifndef LAUNCH_EXT_H
+#define LAUNCH_EXT_H
+
+class wxString;
+
+/**
+ * Launches the given file or folder in the host OS
+ * @param aPath is a path to a file or folder
+ */
+void LaunchExternal( const wxString& aPath );
+
+#endif

include/search_stack.h

@@ -2,7 +2,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2014 CERN
- * Copyright (C) 1992-2014 KiCad Developers, see CHANGELOG.TXT for contributors.
+ * Copyright (C) 1992-2021 KiCad Developers, see AUTHORS.txt for contributors.
  * @author Maciej Suminski <maciej.suminski@cern.ch>
  *
  * This program is free software; you can redistribute it and/or
@@ -70,17 +70,6 @@ public:
     wxString FilenameWithRelativePathInSearchList(
             const wxString& aFullFilename, const wxString& aBaseDir );
 
-    wxString FindValidPath( const wxString& aFileName ) const
-    {
-#if 1   // might not be needed
-
-        if( wxFileName::FileExists( aFileName ) )
-            return aFileName;
-        else
-#endif
-            return wxPathList::FindValidPath( aFileName );
-    }
-
     /**
      * Function AddPaths
      * insert or append path(s)

kicad/CMakeLists.txt

@@ -45,7 +45,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE kicad.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME kicad )
 endif()
 

kicad/mainframe.cpp

@@ -40,6 +40,7 @@
 #include <build_version.h>
 #include <dialog_configure_paths.h>
 #include <dialog_edit_library_tables.h>
+#include <launch_ext.h>
 #include "pgm_kicad.h"
 #include "tree_project_frame.h"
 
@@ -536,21 +537,7 @@ void KICAD_MANAGER_FRAME::OnOpenFileInTextEditor( wxCommandEvent& event )
 void KICAD_MANAGER_FRAME::OnBrowseInFileExplorer( wxCommandEvent& event )
 {
     // open project directory in host OS's file explorer
-    wxString project_dir = Prj().GetProjectPath();
-
-#ifdef __WXMAC__
-    wxString msg;
-
-    // Quote in case there are spaces in the path.
-    msg.Printf( "open \"%s\"", project_dir );
-
-    system( msg.c_str() );
-#else
-    // Quote in case there are spaces in the path.
-    AddDelimiterString( project_dir );
-
-    wxLaunchDefaultApplication( project_dir );
-#endif
+    LaunchExternal( Prj().GetProjectPath() );
 }
 
 

kicad/project_template.cpp

@@ -2,7 +2,7 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2012 Brian Sidebotham <brian.sidebotham@gmail.com>
- * Copyright (C) 1992-2019 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -29,6 +29,8 @@
 #include <wx/wfstream.h>
 #include <wx/log.h>
 
+#include <wildcards_and_files_ext.h>
+
 #include "project_template.h"
 
 
@@ -183,8 +185,10 @@ bool PROJECT_TEMPLATE::CreateProject( wxFileName& aNewProjectPath, wxString* aEr
 
         // Do not rename project specific symbol libraries.  This will break the symbol library
         // table which will cause broken symbol library links in the schematic.
-        if( !( destination.GetExt() == "dcm"
-             || ( destination.GetExt() == "lib" && !destination.GetName().EndsWith( "-cache" ) ) ) )
+        if( !( destination.GetExt() == PageLayoutDescrFileExtension )
+          && !( destination.GetExt() == "dcm"
+              || ( destination.GetExt() == "lib"
+                 && !destination.GetName().EndsWith( "-cache" ) ) ) )
             currname.Replace( basename, aNewProjectPath.GetName() );
 
         destination.SetName( currname );

pagelayout_editor/CMakeLists.txt

@@ -64,7 +64,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE pl_editor.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME pl_editor )
 endif()
 

pcb_calculator/CMakeLists.txt

@@ -50,7 +50,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE pcb_calculator.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME pcb_calculator )
 endif()
 

pcbnew/CMakeLists.txt

@@ -592,7 +592,7 @@ if( APPLE )
         MACOSX_PACKAGE_LOCATION Resources
         )
     set( MACOSX_BUNDLE_ICON_FILE pcbnew.icns )
-    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad-pcb.kicad )
+    set( MACOSX_BUNDLE_GUI_IDENTIFIER org.kicad.kicad )
     set( MACOSX_BUNDLE_NAME pcbnew )
 endif()
 

pcbnew/board_design_settings.cpp

@@ -377,7 +377,7 @@ public:
         wxString                   oldPath = aConfig->GetPath();
         NETCLASSES::const_iterator nc = m_Pt_param->begin();
 
-        for( int index = 0; index <= m_Pt_param->GetCount(); ++index )
+        for( size_t index = 0; index <= m_Pt_param->GetCount(); ++index )
         {
             wxString    path = "";
             NETCLASSPTR netclass;
@@ -495,6 +495,8 @@ BOARD_DESIGN_SETTINGS::BOARD_DESIGN_SETTINGS() :
     m_MicroViasMinSize  = Millimeter2iu( DEFAULT_MICROVIASMINSIZE );
     m_MicroViasMinDrill = Millimeter2iu( DEFAULT_MICROVIASMINDRILL );
 
+    m_HoleToHoleMin = Millimeter2iu( DEFAULT_HOLETOHOLEMIN );
+
     // Global mask margins:
     m_SolderMaskMargin  = Millimeter2iu( DEFAULT_SOLDERMASK_CLEARANCE );
     m_SolderMaskMinWidth = Millimeter2iu( DEFAULT_SOLDERMASK_MIN_WIDTH );
@@ -502,6 +504,7 @@ BOARD_DESIGN_SETTINGS::BOARD_DESIGN_SETTINGS() :
     m_SolderPasteMarginRatio = 0.0;             // Solder paste margin as a ratio of pad size
                                                 // The final margin is the sum of these 2 values
                                                 // Usually < 0 because the mask is smaller than pad
+
     // Layer thickness for 3D viewer
     m_boardThickness = Millimeter2iu( DEFAULT_BOARD_THICKNESS_MM );
 

pcbnew/class_module.cpp

@@ -4,7 +4,7 @@
  * Copyright (C) 2017 Jean-Pierre Charras, jp.charras at wanadoo.fr
  * Copyright (C) 2015 SoftPLC Corporation, Dick Hollenbeck <dick@softplc.com>
  * Copyright (C) 2015 Wayne Stambaugh <stambaughw@gmail.com>
- * Copyright (C) 1992-2019 KiCad Developers, see AUTHORS.txt for contributors.
+ * Copyright (C) 1992-2021 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
@@ -460,7 +460,9 @@ EDA_RECT MODULE::GetFootprintRect() const
 
     area.SetOrigin( m_Pos );
     area.SetEnd( m_Pos );
-    area.Inflate( Millimeter2iu( 0.25 ) );   // Give a min size to the area
+
+    // Give a minimum size to the area in case footprint has no drawing or pad objects.
+    area.Inflate( Millimeter2iu( 0.5 ) );
 
     for( const BOARD_ITEM* item = m_Drawings.GetFirst(); item; item = item->Next() )
     {
@@ -479,13 +481,6 @@ const EDA_RECT MODULE::GetBoundingBox() const
 {
     EDA_RECT area = GetFootprintRect();
 
-    // Add in items not collected by GetFootprintRect():
-    for( const BOARD_ITEM* item = m_Drawings.GetFirst(); item; item = item->Next() )
-    {
-        if( item->Type() != PCB_MODULE_EDGE_T )
-            area.Merge( item->GetBoundingBox() );
-    }
-
     area.Merge( m_Value->GetBoundingBox() );
     area.Merge( m_Reference->GetBoundingBox() );
 
@@ -529,6 +524,7 @@ SHAPE_POLY_SET MODULE::GetBoundingPoly() const
     poly.Append( p );
 
     BOARD* board = GetBoard();
+
     if( board )
     {
         int biggest_clearance = board->GetDesignSettings().GetBiggestClearanceValue();
@@ -650,13 +646,20 @@ bool MODULE::HitTest( const EDA_RECT& aRect, bool aContained, int aAccuracy ) co
     arect.Inflate( aAccuracy );
 
     if( aContained )
+    {
         return arect.Contains( m_BoundaryBox );
+    }
     else
     {
         // If the rect does not intersect the bounding box, skip any tests
         if( !aRect.Intersects( GetBoundingBox() ) )
             return false;
 
+        // If there are no drawing objects, there was still an intersection with the reference
+        // and/or value text.
+        if( m_Pads.GetCount() == 0 && m_Drawings.GetCount() == 0 )
+            return true;
+
         // Determine if any elements in the MODULE intersect the rect
         for( D_PAD* pad = m_Pads; pad; pad = pad->Next() )
         {

pcbnew/class_pad.cpp

@@ -481,15 +481,15 @@ void D_PAD::FlipPrimitives()
 }
 
 
-void D_PAD::MirrorXPrimitives( int aX )
+void D_PAD::MirrorXPrimitives()
 {
     // Mirror custom shapes
     for( unsigned ii = 0; ii < m_basicShapes.size(); ++ii )
     {
         PAD_CS_PRIMITIVE& primitive = m_basicShapes[ii];
 
-        MIRROR( primitive.m_Start.x, aX );
-        MIRROR( primitive.m_End.x, aX );
+        MIRROR( primitive.m_Start.x, 0 );
+        MIRROR( primitive.m_End.x, 0 );
         primitive.m_ArcAngle = -primitive.m_ArcAngle;
 
         switch( primitive.m_Shape )

pcbnew/class_pad.h

@@ -336,11 +336,10 @@ public:
     void FlipPrimitives();
 
     /**
-     * Mirror the primitives about a coordinate
-     *
-     * @param aX the x coordinate about which to mirror
+     * Mirror the primitives around the vertical axis. Primitive coordinates are
+     * relative to the pad, and are not absolute
      */
-    void MirrorXPrimitives( int aX );
+    void MirrorXPrimitives();
 
     /**
      * Import to the basic shape list

pcbnew/class_zone.cpp

@@ -1307,6 +1307,11 @@ bool ZONE_CONTAINER::BuildSmoothedPoly( SHAPE_POLY_SET& aSmoothedPoly ) const
             aSmoothedPoly = m_Poly->Fillet( m_cornerRadius, ARC_HIGH_DEF );
         else
             aSmoothedPoly = m_Poly->Fillet( m_cornerRadius, ARC_LOW_DEF );
+
+        // In some cases the resulting polygon is strange... due to some bug in Fillet()
+        // that happens in (rare) polygon shapes. So clip it with the
+        // initial polygon (in all cases aSmoothedPoly must be contained by m_Poly)
+        aSmoothedPoly.BooleanIntersection( *m_Poly, SHAPE_POLY_SET::PM_FAST );
         break;
 
     default:

pcbnew/drc_clearance_test_functions.cpp

@@ -296,6 +296,8 @@ bool DRC::doTrackDrc( TRACK* aRefSeg, TRACK* aStart, bool aTestPads, bool aTestZ
                 err = false;
             else if( layer1 == F_Cu  &&  layer2 == In1_Cu  )
                 err = false;
+            else if( static_cast<int>( layer2 ) - static_cast<int>( layer1 ) == 1 )
+                err = false;
 
             if( err )
             {

pcbnew/github/github_getliblist.cpp

@@ -217,7 +217,7 @@ bool GITHUB_GETLIBLIST::remoteGetJSON( const std::string& aFullURLCommand, wxStr
     wxLogDebug( wxT( "Attempting to download: " ) + aFullURLCommand );
 
     kcurl.SetURL( aFullURLCommand );
-    kcurl.SetUserAgent( "http://kicad-pcb.org" );
+    kcurl.SetUserAgent( "https://www.kicad.org" );
     kcurl.SetHeader( "Accept", m_option_string );
     kcurl.SetFollowRedirects( true );
 

pcbnew/github/github_plugin.cpp

@@ -582,7 +582,7 @@ void GITHUB_PLUGIN::remoteGetZip( const wxString& aRepoURL )
     KICAD_CURL_EASY kcurl;      // this can THROW_IO_ERROR
 
     kcurl.SetURL( zip_url.c_str() );
-    kcurl.SetUserAgent( "http://kicad-pcb.org" );
+    kcurl.SetUserAgent( "https://www.kicad.org" );
     kcurl.SetHeader( "Accept", "application/zip" );
     kcurl.SetFollowRedirects( true );
 

pcbnew/pcb_draw_panel_gal.cpp

@@ -37,6 +37,7 @@
 #include <class_track.h>
 #include <class_marker_pcb.h>
 #include <pcb_base_frame.h>
+#include "footprint_edit_frame.h"
 #include <confirm.h>
 
 #include <gal/graphics_abstraction_layer.h>
@@ -439,6 +440,20 @@ BOX2I PCB_DRAW_PANEL_GAL::GetDefaultViewBBox() const
     if( m_worksheet && m_view->IsLayerVisible( LAYER_WORKSHEET ) )
         return m_worksheet->ViewBBox();
 
+    // The footprint editor frame has no worksheet, but the best default view box is the
+    // view box able to show the current footprint, even if some layers are not shown
+    if( m_edaFrame->IsType( FRAME_PCB_MODULE_EDITOR ) )
+    {
+        PCB_BASE_FRAME* fp_frame = static_cast<PCB_BASE_FRAME*>( m_edaFrame );
+        MODULE* footprint = fp_frame->GetBoard()->m_Modules;
+
+        if( footprint )
+        {
+            EDA_RECT bbox = footprint->GetBoundingBox();
+            return bbox;
+        }
+    }
+
     return BOX2I();
 }
 

pcbnew/pcb_painter.cpp

@@ -2,6 +2,8 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2013-2019 CERN
+ * Copyright (C) 2021 KiCad Developers, see AUTHORS.txt for contributors.
+ *
  * @author Tomasz Wlostowski <tomasz.wlostowski@cern.ch>
  * @author Maciej Suminski <maciej.suminski@cern.ch>
  *
@@ -383,7 +385,40 @@ bool PCB_PAINTER::Draw( const VIEW_ITEM* aItem, int aLayer )
         return false;
     }
 
-    return true;
+    // Draw bounding boxes after drawing objects so they can be seen.
+#if 0
+    // Show bounding boxes of painted objects for debugging.
+    EDA_RECT box = item->GetBoundingBox();
+    m_gal->SetIsFill( false );
+    m_gal->SetIsStroke( true );
+
+    if( item->Type() == PCB_MODULE_T )
+        m_gal->SetStrokeColor( item->IsSelected() ? COLOR4D( 1.0, 0.2, 0.2, 1 ) :
+                               COLOR4D( MAGENTA ) );
+    else
+        m_gal->SetStrokeColor( item->IsSelected() ? COLOR4D( 1.0, 0.2, 0.2, 1 ) :
+                               COLOR4D( 0.2, 0.2, 0.2, 1 ) );
+
+    m_gal->SetLineWidth( 1.5 / m_gal->GetWorldScale() );
+    m_gal->DrawRectangle( box.GetOrigin(), box.GetEnd() );
+
+    if( item->Type() == PCB_MODULE_T )
+    {
+        m_gal->SetStrokeColor( item->IsSelected() ? COLOR4D( 1.0, 0.2, 0.2, 1 ) :
+                               COLOR4D( CYAN ) );
+
+        const MODULE* fp = static_cast<const MODULE*>( item );
+
+        if( fp )
+        {
+            SHAPE_POLY_SET convex = fp->GetBoundingPoly();
+
+            m_gal->DrawPolyline( convex.COutline( 0 ) );
+        }
+    }
+#endif
+
+return true;
 }
 
 

pcbnew/pcbplot.cpp

@@ -148,11 +148,13 @@ const wxString GetGerberFileFunctionAttribute( const BOARD *aBoard, LAYER_NUM aL
         break;
 
     case B_Fab:
-        attrib = "Other,Fab,Bot";
+        // This is actually a assembly layer
+        attrib = "AssemblyDrawing,Bot";
         break;
 
     case F_Fab:
-        attrib = "Other,Fab,Top";
+        // This is actually a assembly layer
+        attrib = "AssemblyDrawing,Top";
         break;
 
     case B_Cu:

pcbnew/python/kicad_pyshell/__init__.py

@@ -108,6 +108,12 @@ class PcbnewPyShell(editor.EditorNotebookFrame):
         self.setEditor(self.crust.editor)
         self.crust.editor.SetFocus()
 
+        """Keep pydoc output on stdout instead of pager and
+            place the stdout into the editor window """
+        import pydoc
+        self._keep_stdin = sys.stdin
+        pydoc.pager = pydoc.plainpager
+
         self.LoadHistory()
 
     def OnAbout(self, event):

pcbnew/tools/edit_tool.cpp

@@ -761,7 +761,7 @@ static void mirrorPadX( D_PAD& aPad, const wxPoint& aMirrorPoint )
     wxPoint tmpPt = mirrorPointX( aPad.GetPosition(), aMirrorPoint );
 
     if( aPad.GetShape() == PAD_SHAPE_CUSTOM )
-        aPad.MirrorXPrimitives( tmpPt.x );
+        aPad.MirrorXPrimitives();
 
     aPad.SetPosition( tmpPt );
 

pcbnew/tools/selection_tool.cpp

@@ -2,9 +2,10 @@
  * This program source code file is part of KiCad, a free EDA CAD application.
  *
  * Copyright (C) 2013-2017 CERN
+ * Copyright (C) 2018-2021 KiCad Developers, see AUTHORS.txt for contributors.
+ *
  * @author Tomasz Wlostowski <tomasz.wlostowski@cern.ch>
  * @author Maciej Suminski <maciej.suminski@cern.ch>
- * Copyright (C) 2018 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
@@ -1701,9 +1702,14 @@ bool SELECTION_TOOL::selectable( const BOARD_ITEM* aItem, bool checkVisibilityOn
             return false;
 
         // Allow selection of footprints if some part of the footprint is visible.
-
         MODULE* module = const_cast<MODULE*>( static_cast<const MODULE*>( aItem ) );
 
+        // If the footprint has no drawable items other than the reference and value test,
+        // then it can be selected.
+        if( ( module->GraphicalItemsList().GetCount() == 0 ) &&
+            ( module->PadsList().GetCount() == 0 ) )
+            return true;
+
         for( auto item : module->GraphicalItems() )
         {
             if( selectable( item, true ) )

plugins/3d/oce/loadmodel.cpp

@@ -71,6 +71,8 @@
 #include <TDF_LabelSequence.hxx>
 #include <TDF_ChildIterator.hxx>
 
+#include <Standard_Version.hxx>
+
 #include "plugins/3dapi/ifsg_all.h"
 
 // log mask for wxLogTrace
@@ -865,8 +867,10 @@ bool processFace( const TopoDS_Face& face, DATA& data, SGNODE* parent,
     else
         S3D::AddSGNodeRef( vshape.GetRawPtr(), ocolor );
 
+#if OCC_VERSION_HEX < 0x070600
     const TColgp_Array1OfPnt&    arrPolyNodes = triangulation->Nodes();
     const Poly_Array1OfTriangle& arrTriangles = triangulation->Triangles();
+#endif
 
     std::vector< SGPOINT > vertices;
     std::vector< int > indices;
@@ -875,14 +879,22 @@ bool processFace( const TopoDS_Face& face, DATA& data, SGNODE* parent,
 
     for(int i = 1; i <= triangulation->NbNodes(); i++)
     {
+#if OCC_VERSION_HEX < 0x070600
         gp_XYZ v( arrPolyNodes(i).Coord() );
+#else
+        gp_XYZ v( triangulation->Node(i).Coord() );
+#endif
         vertices.push_back( SGPOINT( v.X(), v.Y(), v.Z() ) );
     }
 
     for(int i = 1; i <= triangulation->NbTriangles(); i++)
     {
         int a, b, c;
+#if OCC_VERSION_HEX < 0x070600
         arrTriangles( i ).Get( a, b, c );
+#else
+        triangulation->Triangle(i).Get(a, b, c);
+#endif
         a--;
 
         if( reverse )

resources/linux/appdata/kicad.appdata.xml.in

@@ -67,29 +67,29 @@
 
   <screenshots>
     <screenshot type="default">
-      <image width="1290" height="825">https://kicad-pcb.org/img/screenshots/appstream/kicad.png</image>
+      <image width="1290" height="825">https://kicad.org/img/screenshots/appstream/kicad.png</image>
     </screenshot>
 
     <screenshot>
       <caption>Eeschema Schematic Editor</caption>
-      <image width="1290" height="825">https://kicad-pcb.org/img/screenshots/appstream/eeschema.png</image>
+      <image width="1290" height="825">https://kicad.org/img/screenshots/appstream/eeschema.png</image>
     </screenshot>
 
     <screenshot>
       <caption>PcbNew PCB Layout</caption>
-      <image width="1290" height="825">https://kicad-pcb.org/img/screenshots/appstream/pcbnew.png</image>
+      <image width="1290" height="825">https://kicad.org/img/screenshots/appstream/pcbnew.png</image>
     </screenshot>
 
     <screenshot>
       <caption>PcbNew 3D Viewer</caption>
-      <image width="1290" height="825">https://kicad-pcb.org/img/screenshots/appstream/3dviewer.png</image>
+      <image width="1290" height="825">https://kicad.org/img/screenshots/appstream/3dviewer.png</image>
     </screenshot>
   </screenshots>
 
-  <url type="homepage">https://kicad-pcb.org/</url>
-  <url type="bugtracker">https://kicad-pcb.org/help/report-a-bug/</url>
-  <url type="help">https://docs.kicad-pcb.org/</url>
-  <url type="donation">https://kicad-pcb.org/</url>
+  <url type="homepage">https://www.kicad.org/</url>
+  <url type="bugtracker">https://go.kicad.org/issue</url>
+  <url type="help">https://go.kicad.org/docs/5.1</url>
+  <url type="donation">https://go.kicad.org/donate</url>
 
   <update_contact>kicad-developers@lists.launchpad.net</update_contact>
   <developer_name>The KiCad Developers</developer_name>

resources/macos/plist/bitmap2component.Info.plist.in

@@ -7,7 +7,7 @@
     <key>CFBundleExecutable</key>             <string>bitmap2component</string>
     <key>CFBundleGetInfoString</key>          <string></string>
     <key>CFBundleIconFile</key>               <string>bitmap2component.icns</string>
-    <key>CFBundleIdentifier</key>             <string>org.kicad-pcb.bitmap2component</string>
+    <key>CFBundleIdentifier</key>             <string>org.kicad.bitmap2component</string>
     <key>CFBundleInfoDictionaryVersion</key>  <string>6.0</string>
     <key>CFBundleLongVersionString</key>      <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                   <string>Bitmap 2 Component</string>

resources/macos/plist/eeschema.Info.plist.in

@@ -19,7 +19,7 @@
     <key>CFBundleExecutable</key>                   <string>eeschema</string>
     <key>CFBundleGetInfoString</key>                <string></string>
     <key>CFBundleIconFile</key>                     <string>eeschema.icns</string>
-    <key>CFBundleIdentifier</key>                   <string>org.kicad-pcb.eeschema</string>
+    <key>CFBundleIdentifier</key>                   <string>org.kicad.eeschema</string>
     <key>CFBundleInfoDictionaryVersion</key>        <string>6.0</string>
     <key>CFBundleLongVersionString</key>            <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                         <string>Eeschema</string>

resources/macos/plist/gerbview.Info.plist.in

@@ -39,7 +39,7 @@
     <key>CFBundleExecutable</key>                   <string>gerbview</string>
     <key>CFBundleGetInfoString</key>                <string></string>
     <key>CFBundleIconFile</key>                     <string>gerbview.icns</string>
-    <key>CFBundleIdentifier</key>                   <string>org.kicad-pcb.gerbview</string>
+    <key>CFBundleIdentifier</key>                   <string>org.kicad.gerbview</string>
     <key>CFBundleInfoDictionaryVersion</key>        <string>6.0</string>
     <key>CFBundleLongVersionString</key>            <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                         <string>GerbView</string>

resources/macos/plist/kicad.Info.plist.in

@@ -18,7 +18,7 @@
     <key>CFBundleExecutable</key>                   <string>kicad</string>
     <key>CFBundleGetInfoString</key>                <string></string>
     <key>CFBundleIconFile</key>                     <string>kicad.icns</string>
-    <key>CFBundleIdentifier</key>                   <string>org.kicad-pcb.kicad</string>
+    <key>CFBundleIdentifier</key>                   <string>org.kicad.kicad</string>
     <key>CFBundleInfoDictionaryVersion</key>        <string>6.0</string>
     <key>CFBundleLongVersionString</key>            <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                         <string>KiCad</string>

resources/macos/plist/pcb_calculator.Info.plist.in

@@ -6,7 +6,7 @@
     <key>CFBundleExecutable</key>              <string>pcb_calculator</string>
     <key>CFBundleGetInfoString</key>           <string></string>
     <key>CFBundleIconFile</key>                <string>pcb_calculator.icns</string>
-    <key>CFBundleIdentifier</key>              <string>org.kicad-pcb.pcb_calculator</string>
+    <key>CFBundleIdentifier</key>              <string>org.kicad.pcb_calculator</string>
     <key>CFBundleInfoDictionaryVersion</key>   <string>6.0</string>
     <key>CFBundleLongVersionString</key>       <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                    <string>PCB Calculator</string>

resources/macos/plist/pcbnew.Info.plist.in

@@ -21,7 +21,7 @@
     <key>CFBundleExecutable</key>               <string>pcbnew</string>
     <key>CFBundleGetInfoString</key>            <string></string>
     <key>CFBundleIconFile</key>                 <string>pcbnew.icns</string>
-    <key>CFBundleIdentifier</key>               <string>org.kicad-pcb.pcbnew</string>
+    <key>CFBundleIdentifier</key>               <string>org.kicad.pcbnew</string>
     <key>CFBundleInfoDictionaryVersion</key>    <string>6.0</string>
     <key>CFBundleLongVersionString</key>        <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                     <string>Pcbnew</string>

resources/macos/plist/pleditor.Info.plist.in

@@ -19,7 +19,7 @@
     <key>CFBundleExecutable</key>                   <string>pl_editor</string>
     <key>CFBundleGetInfoString</key>                <string></string>
     <key>CFBundleIconFile</key>                     <string>pl_editor.icns</string>
-    <key>CFBundleIdentifier</key>                   <string>org.kicad-pcb.pl_editor</string>
+    <key>CFBundleIdentifier</key>                   <string>org.kicad.pl_editor</string>
     <key>CFBundleInfoDictionaryVersion</key>        <string>6.0</string>
     <key>CFBundleLongVersionString</key>            <string>@KICAD_VERSION_FULL@</string>
     <key>CFBundleName</key>                         <string>Page Layout Editor</string>