Page MenuHome
Differential D12076

Cleanup: Node Wrangler: Remove assignments in nodes_at_pos
ClosedPublic
Actions

Authored by jim man (jimman2003) on Jul 29 2021, 2:29 PM.

Details

Reviewers
Jacques Lucke (JacquesLucke)
Commits
rBA64eeb14bf27d: Node Wrangler: remove unnecessary code
Summary

Remove some unessesary (according to my Python LSP) assignment in nodes_at_pos.

Diff Detail

Event Timeline

jim man (jimman2003) requested review of this revision.Jul 29 2021, 2:29 PM
jim man (jimman2003) created this revision.
jim man (jimman2003) added a reviewer: Bastien Montagne (mont29).Jul 29 2021, 7:30 PM
jim man (jimman2003) removed a reviewer: Bastien Montagne (mont29).Aug 6 2021, 4:50 AM
Pratik Borhade (PratikPB2123) edited projects, added Add-ons (Community); removed Add-ons (BF-Blender).Aug 17 2021, 11:18 AM
Jacques Lucke (JacquesLucke) accepted this revision.Aug 26 2021, 12:53 PM
Jacques Lucke (JacquesLucke) added a subscriber: Jacques Lucke (JacquesLucke).

Nice, some node wrangler cleanup. The addon could use soo much more cleanup in general..

This revision is now accepted and ready to land.Aug 26 2021, 12:53 PM
Jacques Lucke (JacquesLucke) added a comment.Aug 26 2021, 12:55 PM

Quick note, you should config your editor so that it removes trailing whitespace (line 742 has trailing whitespace).

Closed by commit rBA64eeb14bf27d: Node Wrangler: remove unnecessary code (authored by jim man (jimman2003), committed by Jacques Lucke (JacquesLucke)). · Explain WhyAug 26 2021, 12:57 PM
This revision was automatically updated to reflect the committed changes.
Jacques Lucke (JacquesLucke) added a commit: rBA64eeb14bf27d: Node Wrangler: remove unnecessary code.
Pratik Borhade (PratikPB2123) awarded a token.Aug 26 2021, 1:02 PM

Revision Contents
Changeset List

PathSize
5 lines

Diff 40117

View Options

node_wrangler.py

Loading...