[NetlistBrowser] Fix extra click issue with instances (#1229)

* [Netlist Browser] GUI Widget to browse a loaded netlist

Signed-off-by: Chris Lavin <[email protected]>

* Address review comments

Signed-off-by: Chris Lavin <[email protected]>

* Fix bug on second click of instances

Signed-off-by: Chris Lavin <[email protected]>

* Adding NetlistBrowser to import list in Jython

Signed-off-by: Chris Lavin <[email protected]>

* [NetlistBrowser] Exit with 1 if wrong # args

Signed-off-by: Eddie Hung <[email protected]>

---------

Signed-off-by: Chris Lavin <[email protected]>
Signed-off-by: Eddie Hung <[email protected]>
Co-authored-by: Eddie Hung <[email protected]>

Author: clavin-xlnx

src/com/xilinx/rapidwright/gui/NetlistBrowser.java

@@ -109,6 +109,7 @@ public static void main(String[] args) {
 
         if (args.length != 1) {
             System.out.println("USAGE: <input.dcp|input.edf>");
+            System.exit(1);
         }
 
         EDIFNetlist netlist = null;

src/com/xilinx/rapidwright/gui/NetlistTreeWidget.java

@@ -60,6 +60,9 @@ public NetlistTreeWidget(String header, EDIFNetlist netlist) {
     }
 
     public QTreeWidgetItem populateCellInst(QTreeWidgetItem curr, EDIFHierCellInst inst) {
+        if (curr.childCount() > 1) {
+            return curr;
+        }
         // Remove DUMMY
         curr.removeChild(curr.child(0));
 

src/com/xilinx/rapidwright/util/Jython.java

@@ -162,6 +162,7 @@ public static void main(String[] args) {
                     com.xilinx.rapidwright.design.DesignTools.class,
                     com.xilinx.rapidwright.design.tools.LUTTools.class,
                     com.xilinx.rapidwright.device.helper.TileColumnPattern.class,
+                    com.xilinx.rapidwright.gui.NetlistBrowser.class,
             };
 
             args = new String[3];