pcbnew - add tracks length for netlist inspector

Bug #1817272 reported by Andrey Fedorushkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Committed
Wishlist
Unassigned

Bug Description

Hi all

This patch add tracks length and export netlist to txt file in csv format for netlist inspector.
I use it to control the alignment of high-speed lines.

Revision history for this message
Andrey Fedorushkov (faa) wrote :
Revision history for this message
Seth Hillbrand (sethh) wrote :

I'll set this to v6 so that we can look at merging as soon as the window opens. At first glance, I see no issues with this patch. Anyone else?

Changed in kicad:
importance: Undecided → Wishlist
milestone: none → 6.0.0-rc1
status: New → Triaged
Revision history for this message
Andrey Fedorushkov (faa) wrote :

This patch for the current master (5.1.0-rc). This is a functional from the branch kicad-gost-committers.

Revision history for this message
Jeff Young (jeyjey) wrote :

Master is now open for 6.0.

@Andrey, can you rebase the patch on the current master so we can merge it? Thanks, Jeff.

Revision history for this message
Andrey Fedorushkov (faa) wrote :

Ok.
patch rebase for current master

Revision history for this message
Jeff Young (jeyjey) wrote :

@Andrey, can you recreate the patch using git format-patch? If I commit a diff-based patch it will go in under my name rather than yours.

Revision history for this message
Andrey Fedorushkov (faa) wrote :
Revision history for this message
Jeff Young (jeyjey) wrote :

@Andrey, I've merged you patch. Thanks for your contribution to Kicad!

Changed in kicad:
status: Triaged → Fix Committed
Revision history for this message
Wayne Stambaugh (stambaughw) wrote :

@Jeff, there were a few coding policy violations (trailing white space, missing spaces before and after braces, and line length greater than 99 columns) in the patch. Please do not hesitate to ask the patch submitter to fix formatting issues. Now that we have a commit hook for clang-format, getting the formatting correct is fairly trivial.

Revision history for this message
Jeff Young (jeyjey) wrote :

Will do. (In the meantime I've fixed up the code policy issues.)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.