Browse Source

Fixed compilation warnings

pull/7/merge
Tomasz Włostowski 9 years ago
parent
commit
113163257d
  1. 1
      pcbnew/board_netlist_updater.cpp
  2. 1
      pcbnew/board_netlist_updater.h
  3. 10
      pcbnew/dialogs/dialog_global_deletion.cpp
  4. 2
      pcbnew/tools/drawing_tool.cpp
  5. 4
      pcbnew/tools/pcb_tool.cpp

1
pcbnew/board_netlist_updater.cpp

@ -51,7 +51,6 @@
BOARD_NETLIST_UPDATER::BOARD_NETLIST_UPDATER( PCB_EDIT_FRAME* aFrame, BOARD* aBoard ) :
m_commit( aFrame ),
m_frame( aFrame ),
m_board( aBoard )
{
m_reporter = &NULL_REPORTER::GetInstance();

1
pcbnew/board_netlist_updater.h

@ -139,7 +139,6 @@ private:
bool testConnectivity( NETLIST& aNetlist );
BOARD_COMMIT m_commit;
PCB_EDIT_FRAME* m_frame;
BOARD* m_board;
REPORTER* m_reporter;

10
pcbnew/dialogs/dialog_global_deletion.cpp

@ -160,16 +160,16 @@ void DIALOG_GLOBAL_DELETION::AcceptPcbDelete()
masque_layer &= layers_filter;
for( auto item : pcb->Drawings() )
for( auto dwg : pcb->Drawings() )
{
KICAD_T type = item->Type();
LAYER_NUM layer = item->GetLayer();
KICAD_T type = dwg->Type();
LAYER_NUM layer = dwg->GetLayer();
if( delAll
|| ( type == PCB_LINE_T && delDrawings && masque_layer[layer] )
|| ( type == PCB_TEXT_T && delTexts && del_text_layers[layer] ) )
{
commit.Remove( item );
commit.Remove( dwg );
}
}
}
@ -238,6 +238,6 @@ void DIALOG_GLOBAL_DELETION::AcceptPcbDelete()
if( gen_rastnest )
m_Parent->Compile_Ratsnest( NULL, true );
// There is a chance that some of tracks have changed their nets, so rebuild ratsnest from scratch
// There is a chance that some of tracks have changed their nets, so rebuild ratsnest from scratch
m_Parent->GetCanvas()->Refresh();
}

2
pcbnew/tools/drawing_tool.cpp

@ -1413,8 +1413,6 @@ int DRAWING_TOOL::DrawVia( const TOOL_EVENT& aEvent )
}
}
printf( "Found zones : %d\n", foundZones.size() );
std::sort( foundZones.begin(), foundZones.end(),
[] ( const ZONE_CONTAINER* a, const ZONE_CONTAINER* b ) {
return a->GetLayer() < b->GetLayer();

4
pcbnew/tools/pcb_tool.cpp

@ -149,10 +149,10 @@ void PCB_TOOL::doInteractiveItemPlacement( INTERACTIVE_PLACER_BASE *aPlacer,
if ( aOptions & IPO_SINGLE_CLICK )
{
VECTOR2I cursorPos = controls()->GetCursorPosition();
VECTOR2I pos = controls()->GetCursorPosition();
newItem = aPlacer->CreateItem();
newItem->SetPosition( wxPoint( cursorPos.x, cursorPos.y ) );
newItem->SetPosition( wxPoint( pos.x, pos.y ) );
preview.Add( newItem.get() );
}

Loading…
Cancel
Save