electron-rare/kicad-source-mirror
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Clean up TODO.

Browse Source 
While it might be tempting to put these in the individual property
setters, there's no well-defined order in which they're called,
and many of the consistency checks are between multiple properties.
This commit is contained in:
Jeff Young
2026-01-31 15:13:29 +00:00
parent 239a0fe745
commit 7da3f06ca6
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pcbnew/padstack.cpp
+4 -3
View File
@@ -99,9 +99,10 @@ PADSTACK& PADSTACK::operator=( const PADSTACK &aOther )
m_frontPostMachining = aOther.m_frontPostMachining;
m_backPostMachining = aOther.m_backPostMachining;
// Data consistency enforcement logic that used to live in the pad properties dialog
// TODO(JE) Should these move to individual property setters, so that they are always
// enforced even through the properties panel and API?
// Data consistency enforcement logic that used to live in the pad properties dialog.
// While it might be tempting to put these in the individual property setters, there's no
// well-defined order in which they're called, and many of the consistency checks are
// between multiple properties.
ForEachUniqueLayer(
[&]( PCB_LAYER_ID aLayer )