This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author terry.reedy
Recipients Jim.Jewett, can.ibanoglu, markroseman, python-dev, terry.reedy
Date 2015-08-19.19:23:24
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1440012205.58.0.0611172378469.issue24790@psf.upfronthosting.co.za>
In-reply-to
Content
NEWS patches should be omitted from submitted .diff or .patch files because a) NEWS is so volatile that merge conflicts applying the .diff are likely; b) NEWS is different between versions, so that forward merge conflicts are possible (usual for Idle items, so I apply them separately) and b) committers tend to rewrite then anyway.  I will later prepare and separately commit one entry for this issue.

ACKS is more stable within and between versions, so including a acks patch is ok.  But then the patch must be prepared for 3.4 and 2.7 specifically, and I want to know ahead of time if there will be a 3.4 to 3.5 or 3.5 to 3.6 merge conflict.  You can include it or leave it out (easier).

I tested both the .patch and diff in msg248849.  When putting code or diffs in messages, please include function/class/method names or lineno.  With this patch, SubList always return a list of one item.  Note that we can always use frame.f_locals, whether it is or is not f_globals.

I am not exactly sure what you meant by your your last question, but it lead me to the answer.  When a tree item with + is clicked, the corresponding GetSubList method is called. What we want FrameTreeItem.GetSubList to do, instead of returning a list of (now 1) VariablesTreeItem, is to return what VariablesTreeItem.GetSubList would (when clicked subequently).  Replace most of the body of the former (FrameTI.GSL) with the body of the latter (VarTI.GSL), with 'self.object' replaced by 'frame.f_locals'.  Class VariablesTreeItem will then be obsolete.
History
Date User Action Args
2015-08-19 19:23:25terry.reedysetrecipients: + terry.reedy, markroseman, python-dev, Jim.Jewett, can.ibanoglu
2015-08-19 19:23:25terry.reedysetmessageid: <1440012205.58.0.0611172378469.issue24790@psf.upfronthosting.co.za>
2015-08-19 19:23:25terry.reedylinkissue24790 messages
2015-08-19 19:23:24terry.reedycreate