Debug spew in BOARD_CONNECTED_ITEM (lp:#1824587)
Original report created by John Beard (john-j-beard)
When placing a component with the "O" tool in pcbnew, in a Debug build, a
lot of DBG spew is produced when the as-yet-unplaced component is moved
around:
* board_connected_item.cpp 97:
DBG(printf( "%s: NULL netclass,type %d", __func__, Type() );)
* board_connected_item.cpp 115:
DBG(printf( "%s: NULL board,type %d", __func__, Type() );)
I assume these warnings are spurious, as it doesn't seem to be causing
issues. However, spamming the console with strings (especially ones without
\n!) is pretty annoying - hundreds of copies of:
GetNetClass: NULL board,type 4GetClearance: NULL netclass,type
4GetNetClass: NULL board,type 4GetClearance: NULL netclass,type
4GetNetClass: NULL board,type 4GetClearance: NULL netclass,type
4GetNetClass: NULL board,type 4GetClearance: NULL netclass,type
4GetNetClass: NULL board,type 4
Can we just remove the debug and keep the behaviour, or should we be
guarding this at a higher level? They've been there since 2014, so this
probably affects 5.1 as well (but I don't have a Debug build of that ATM)
Original tags: pcbnew