Fix minor Coverity warnings

This commit is contained in:
jean-pierre charras 2022-08-16 18:54:20 +02:00
parent 742757b98b
commit 9e3e4ae9a6
2 changed files with 11 additions and 4 deletions

View File

@ -452,6 +452,10 @@ ASCH_HARNESS_CONNECTOR::ASCH_HARNESS_CONNECTOR( const std::map<wxString, wxStrin
Color = ALTIUM_PARSER::ReadInt( aProps, "COLOR", 0 ); Color = ALTIUM_PARSER::ReadInt( aProps, "COLOR", 0 );
AreaColor = ALTIUM_PARSER::ReadInt( aProps, "AREACOLOR", 0 ); AreaColor = ALTIUM_PARSER::ReadInt( aProps, "AREACOLOR", 0 );
IndexInSheet = 0;
LineWidth = 0;;
LocationPrimaryConnectionPosition = 0;
} }
@ -477,6 +481,7 @@ ASCH_HARNESS_ENTRY::ASCH_HARNESS_ENTRY( const std::map<wxString, wxString>& aPro
AreaColor = ALTIUM_PARSER::ReadInt( aProps, "AREACOLOR", 0 ); AreaColor = ALTIUM_PARSER::ReadInt( aProps, "AREACOLOR", 0 );
TextColor = ALTIUM_PARSER::ReadInt( aProps, "TEXTCOLOR", 0 ); TextColor = ALTIUM_PARSER::ReadInt( aProps, "TEXTCOLOR", 0 );
TextFontID = ALTIUM_PARSER::ReadInt( aProps, "TEXTFONTID", 0 ); TextFontID = ALTIUM_PARSER::ReadInt( aProps, "TEXTFONTID", 0 );
TextStyle = 0;
} }
@ -712,7 +717,7 @@ ASCH_SHEET_FONT::ASCH_SHEET_FONT( const std::map<wxString, wxString>& aProps, in
Italic = ALTIUM_PARSER::ReadBool( aProps, "ITALIC" + sid, false ); Italic = ALTIUM_PARSER::ReadBool( aProps, "ITALIC" + sid, false );
Bold = ALTIUM_PARSER::ReadBool( aProps, "BOLD" + sid, false ); Bold = ALTIUM_PARSER::ReadBool( aProps, "BOLD" + sid, false );
Underline = ALTIUM_PARSER::ReadBool( aProps, "UNDERLINE" + sid, false ); Underline = ALTIUM_PARSER::ReadBool( aProps, "UNDERLINE" + sid, false );
AreaColor = ALTIUM_PARSER::ReadInt( aProps, "AREACOLOR" + sid, 0 ); AreaColor = ALTIUM_PARSER::ReadInt( aProps, "AREACOLOR" + sid, 0 );
} }

View File

@ -130,6 +130,8 @@ SCH_ALTIUM_PLUGIN::SCH_ALTIUM_PLUGIN()
m_rootSheet = nullptr; m_rootSheet = nullptr;
m_currentSheet = nullptr; m_currentSheet = nullptr;
m_schematic = nullptr; m_schematic = nullptr;
m_harnessOwnerIndexOffset = 0;
m_harnessEntryParent = 0;
m_reporter = &WXLOG_REPORTER::GetInstance(); m_reporter = &WXLOG_REPORTER::GetInstance();
} }
@ -383,7 +385,7 @@ void SCH_ALTIUM_PLUGIN::ParseAdditional( const ALTIUM_COMPOUND_FILE& aAltiumSchF
ParseHarnessEntry( properties ); ParseHarnessEntry( properties );
break; break;
case ALTIUM_SCH_RECORD::HARNESS_TYPE: case ALTIUM_SCH_RECORD::HARNESS_TYPE:
ParseHarnessType( properties ); ParseHarnessType( properties );
break; break;
case ALTIUM_SCH_RECORD::SIGNAL_HARNESS: case ALTIUM_SCH_RECORD::SIGNAL_HARNESS:
ParseSignalHarness( properties ); ParseSignalHarness( properties );
@ -2169,9 +2171,9 @@ void SCH_ALTIUM_PLUGIN::ParsePort( const ASCH_PORT& aElem )
//{ //{
// label = new SCH_HIERLABEL( elem.location + m_sheetOffset, elem.name ); // label = new SCH_HIERLABEL( elem.location + m_sheetOffset, elem.name );
//} //}
label = new SCH_GLOBALLABEL( position, aElem.Name ); label = new SCH_GLOBALLABEL( position, aElem.Name );
switch( aElem.IOtype ) switch( aElem.IOtype )
{ {
default: default: