Terry J. Reedy added the comment:
I guess rev #s are different on different systems. Try d55d1cbf5f9a or
revd55d1cbf5f9a
--
___
Python tracker
<http://bugs.python.org/issue21
Terry J. Reedy added the comment:
Line 1.20 in the correct link.
--
___
Python tracker
<http://bugs.python.org/issue21654>
___
___
Python-bugs-list mailin
Terry J. Reedy added the comment:
This is 2.7 only. Warnings may have been expanded a bit in 2.7.7. I know there
are plans to widen net further in 2.7.8. We can leave this open until you are
convinced it works or discover otherwise.
--
___
Python
Terry J. Reedy added the comment:
Unless you feel like struggling with the 2.7's custom calltip calculation code
for possibly little or no gain, please close this as wont' fix. See #20122,
especially msg208454 and point 3. f6f2d9d04cd0 made this change to improve the
calltip for
Terry J. Reedy added the comment:
Coverage 100%. 2.7 version was easy, so committed after a few minor changes. We
can revisit using the mock later.
--
resolution: -> fixed
stage: needs patch -> resolved
status: open -> closed
___
Pytho
Terry J. Reedy added the comment:
The use of .__new__ was cute. Unfortunately, it did not backport to 2.7 because
tkinter classes were never upgraded from old to new in 2.7 and old-style
classes do not have .__new__. So I monkeypatched the module instead, which is a
but clumbsier than
Changes by Terry J. Reedy :
--
versions: -Python 3.2, Python 3.3
___
Python tracker
<http://bugs.python.org/issue7676>
___
___
Python-bugs-list mailing list
Unsub
New submission from Terry J. Reedy:
Example that prompted this idea due to difficulty of scanning results. The code
lines are relatively long and the hit range from first to (almost) last word in
the line.
Searching 'future' in F:\Python\dev\2\py27\*.c...
...
F:\Python\dev\2\py27\M
Terry J. Reedy added the comment:
I verified that is used in 2.7 for both unbound and
bound methods of both old- and new-style classes. In 3.0, old-style classes and
unbound methods were removed. 2.x types seem not to have __bool__, so I suspect
that the condition code special cases numbers
New submission from Terry J. Reedy:
I miss this from Notepad++. This is essentially Find in Files limited to one
file, without the file name repeated on each line.
Notepad++ puts multiple findall results in one window, with +- marker to expand
or contract a group. It also has findall in all
Terry J. Reedy added the comment:
Whoops. Zach, did you catch that by reading the checkin, running the test, or
seeing a buildbot problem. Is not the first, what symptom on what system
revealed the omission?
--
___
Python tracker
<h
Changes by Terry J. Reedy :
--
Removed message: http://bugs.python.org/msg219852
___
Python tracker
<http://bugs.python.org/issue18292>
___
___
Python-bugs-list m
Terry J. Reedy added the comment:
Whoops. Zach, did you catch that by reading the checkin, running the test, or
seeing a buildbot problem. Is not the first, what symptom on what system
revealed the omission?
--
nosy: +zach.ware
___
Python tracker
Terry J. Reedy added the comment:
Since all tests create a widget with widgets (or destroy it), the new patch
moves root to class scope and simplifies the code a bit. It also subclasses
TextViewer instead of monkey-patching it. Modules have to be monkey-patched
because they cannot be
Terry J. Reedy added the comment:
Tests that need at most tkinter Variables and message boxes run without gui
since those are easy to mock and we want to set and retrieve values on MBox.
Tests of displayed widgets other than Text are gui tests (unless we were to
developed a substantial tk
Terry J. Reedy added the comment:
Why do you think this is a bug? What behavior both looks wrong and gets
improved by the change?
--
nosy: +terry.reedy, tim.peters
stage: -> test needed
___
Python tracker
<http://bugs.python.org/issu
Terry J. Reedy added the comment:
This has been proposed, discussed on both python-list and python-ideas, and
rejected more than once because there are there are multiple possible response
to multiple keys: keep first value, keep second value (.update), keep both (in
a list), or keep neither
Terry J. Reedy added the comment:
I ran the patched code but have not looked at the code itself. Pending such a
look, I would consider committing it if we cannot do better, as it solves both
issues. However, there are two visual issues.
1. The minor one: The blue label does not have drop
Terry J. Reedy added the comment:
This concerns me. I expect that we will eventually want to test a live
EditorWindow or subclass. It appears that root.destroy does not clear all the
widgets created by EditorWindow(root=root). My guess it that something is
created without passing in root, so
Terry J. Reedy added the comment:
The 3.4 stable buildbots are green except for two that ran test_idle ok.
The problem is that in 2.7, unittest.loader does not catch ResourceDenied at
module level whereas is does in 3.4. The only indication that there should be a
difference is that the 3.x doc
Terry J. Reedy added the comment:
Thank you for the investigation. Unless there is a problem in 2.7, which you
have not mentioned, this should be closed as out-of-date, or maybe fixed.
--
___
Python tracker
<http://bugs.python.org/issue18
Terry J. Reedy added the comment:
When writing the human-verified tests in idle_test.htest.py, we discovered that
there is a different between Windows and Linux in focus shifting when opening
an editor window from a visible root window. On Widows, I have to click on the
editor window to enter
New submission from Terry J. Reedy:
Reproducer: On Windows, Open Idle and editor. In editor grep (alt-f3), for
instance, 'print' in /Lib/*.py. While hits are flashing by, close the output
window with [x]
2.7.6 or .7: Output window closes, Idle continues as desired.
3.3.5 or 3.4.1
New submission from Terry J. Reedy:
Spinoff of #12274 and a dependency thereof: new test_configurations.py should
statically test that configparser, called from idlelib.configHandler, can
process all the configuration.def files without error.
--
assignee: terry.reedy
messages: 220053
Terry J. Reedy added the comment:
This issue needs a unittest test added within the framework. I opened #21696 as
a dependency. Since configx.def files can be edited, and, I believe, at least
one is intended to be edited, configHandler should perhaps catch exceptions and
display in a
Terry J. Reedy added the comment:
Add to my list of patches to review.
--
versions: +Python 3.4, Python 3.5 -Python 3.2, Python 3.3
___
Python tracker
<http://bugs.python.org/issue4
Terry J. Reedy added the comment:
I am on Windows 7 also. Did you open the shell from the console interpreter
with import? Run
python -m idlelib.idle_test.htest
and for me the tests that open a separate editor window show the same problem
of no cursor or focus in the editor. Perhaps you ran
Terry J. Reedy added the comment:
There already is an idlelib/idle.bat that starts the corresponding python with
set CURRDIR=%~dp0
start "IDLE" "%CURRDIR%..\..\pythonw.exe" "%CURRDIR%idle.pyw" %1 %2 %3 %4 %5 %6
%7 %8 %9
Arguments added to the .bat command are
Terry J. Reedy added the comment:
Yes, early skipping was the reason I put the test where I did. The simple
change occurred to me today. I have not decided yet whether to also change the
3.4/5 files to match and how to edit the testing README. For one test, I would
not care either way. But I
Terry J. Reedy added the comment:
Another issue for sometests, but which might be fixed in htest.run, is to force
focus to the new widget window opened by the Test_xyz button.
--
___
Python tracker
<http://bugs.python.org/issue21
Terry J. Reedy added the comment:
It seems that I have been over-optimistic about uniformity of behavior. With
the focus issue different for opening Idle and opening subwindows in htest, I
added it to the issue about refining htest #21624.
Given Mark's report, I must rebuild tcl/tk and
Terry J. Reedy added the comment:
Checking the buildbot just now, there is some other stuff after test_tk on X86
Windows 7, but I ran the command you gave (but with python_d) and it passed ok.
So I presume this is really fixed and should stay closed
Changes by Terry J. Reedy :
--
resolution: -> works for me
stage: -> resolved
status: open -> closed
___
Python tracker
<http://bugs.python.or
Terry J. Reedy added the comment:
Thanks for the correction.
--
___
Python tracker
<http://bugs.python.org/issue15780>
___
___
Python-bugs-list mailing list
Unsub
Terry J. Reedy added the comment:
Mark, they are both inactive at the moment.
There are a number of issue around configuration, especially key bindings. I
think the whole process needs review and redoing. For instance, I understand
Roger as saying that the problematical known-invalid list
Terry J. Reedy added the comment:
Looks good on reading. When I get my repository copy repaired, I will test and
either apply or report back. In 2.7 Lib/*.py, 9 module use 'kwds' and 28 use
'kwargs', so I will accept that change.
--
assignee: -> terry.re
Changes by Terry J. Reedy :
--
assignee: kbk ->
versions: +Python 2.7, Python 3.4, Python 3.5 -Python 3.3
___
Python tracker
<http://bugs.python.org/iss
Terry J. Reedy added the comment:
David, thank you for the research on related issues. #5233 explicitly includes
this proposal: "The former effect of -s would now be the default,". So I am
closing this as a partial duplicate. I will explain here why I reject this.
--
Terry J. Reedy added the comment:
This is at least 2 issues: startup and restart. I closed #8378 as a partial
duplicate of the startup issue.
For startup, I reject the idea of changing the default and the meaning of -s.
First, it would break the general back-compatibility policy and possibly
Changes by Terry J. Reedy :
--
stage: needs patch -> patch review
versions: +Python 3.4, Python 3.5 -Python 3.2, Python 3.3
___
Python tracker
<http://bugs.python.org/issu
Terry J. Reedy added the comment:
In 3.4, the only Windows-specific settings I see in config-main.def is
[Keys]
default= 1
name= IDLE Classic Windows
I have presumed that this is somehow changed on other systems. If not, that is
an issue.
Config-keys.def actions (should) have a Meta variant
Terry J. Reedy added the comment:
I decided that anyone with a use case for a bare interpreter, if there is such
a person, would and should be using the console, and that a closed issue is
sufficient documentation for now.
--
resolution: -> wont fix
stage: test needed -> re
Terry J. Reedy added the comment:
Since #20580 has a much more specific plan, I am closing this one as a
duplicate and back-referencing this from there.
--
resolution: -> duplicate
status: open -> closed
superseder: -> IDLE should support platform-specific default config
Terry J. Reedy added the comment:
I closed #1517993 in favor of this.
--
___
Python tracker
<http://bugs.python.org/issue20580>
___
___
Python-bugs-list mailin
Terry J. Reedy added the comment:
I added try: except: and tested on installed 3.4.1, which previously failed.
There is no way that I know of to start repository Idle without a console to
print a traceback to. I added a missing import, removed an incorrect comment,
added others, and changed
Terry J. Reedy added the comment:
The fact that Tab indents regions as well as lines seems not be documented.
This should be added to Automatic Indentation (which should just be
Indentation) with BackTab doc addition if not sooner.
On Windows, 3.4.1, shift-tab is the same as tab. The same is
Terry J. Reedy added the comment:
Tal could have. Anyway, I made a note to look at this or #1252 if I want to
understand Percolator or Delegator or think about changing them.
--
resolution: -> wont fix
status: open -> closed
___
Python t
Changes by Terry J. Reedy :
--
versions: +Python 3.4, Python 3.5 -Python 2.6, Python 3.1
___
Python tracker
<http://bugs.python.org/issue12989>
___
___
Python-bug
Terry J. Reedy added the comment:
Mark, you did not specify which of the variations you tried ;-) I reproduce as
I said in my last message.
C:\Programs\Python34>pythonw -m idlelib
Python 3.4.1 (v3.4.1:c0e311e010fc, May 18 2014, 10:45:13) [
>>> import subprocess
>>>
Terry J. Reedy added the comment:
I updated the patch to match a change, since you posted this, from 70 to 72 in
.def and 'maxformatwidth' to ' limit' in formatparagraph. I also added a
'guard' value of 72 in case GetOption returns None (as it did before I change
Terry J. Reedy added the comment:
The format paragraph entry in main.def is being moved to extensions.def, so
don't test for it in main.def.
--
nosy: +jesstess
___
Python tracker
<http://bugs.python.org/is
Terry J. Reedy added the comment:
Backwards from default was the old, svn way. We hg we merge forward within 3.x.
"ready to commit to 3.4 and merge to 3.5.". If you started with 3.5, then you
would have to do a null merge of the 3.4 patch into 3.5. It is occasionally
done when peo
Terry J. Reedy added the comment:
Refinement 1: in doing coverage for UndoDelegator, I noticed that the htest
function is counted as missing (uncovered). Both of the following in
.coveragerc work to ignore the block: name prefix; comment suffix.
def htest_.*:
.*# htest #
The
Terry J. Reedy added the comment:
cls.percolator.close() cut leaks in half.
test_idle leaked [237, 237, 237, 237] references, sum=948
test_idle leaked [95, 97, 97, 97] memory blocks, sum=386
test_idle leaked [130, 130, 130, 130] references, sum=520
test_idle leaked [60, 62, 62, 62] memory
Terry J. Reedy added the comment:
Great. I will review the next patch (with the change in the comment) tomorrow.
The ttk theme changed message comes up often. I believe it is specific to debug
builds. Perhaps you could keep track of when it occurs so we can make a guess
at the cause. I
Terry J. Reedy added the comment:
"For flowing long blocks of text with fewer structural restrictions (docstrings
or comments), the line length should be limited to 72 characters."
--
___
Python tracker
<http://bugs.python.o
Changes by Terry J. Reedy :
--
title: Idle: test syntax of configuration files -> Idle: test configuration
files
___
Python tracker
<http://bugs.python.org/issu
Terry J. Reedy added the comment:
I am not completely sure what you are asking. Config-keys.def has 4 Idle
Classic sections, one each for Windows, Unix, Mac (still needed?), and OSX. On
my Windows install, the one for Windows is preselected. I have presumed the the
appropriate selections are
Terry J. Reedy added the comment:
'yes!' meant please do so.
--
___
Python tracker
<http://bugs.python.org/issue20577>
___
___
Python-bugs-list mailing
Terry J. Reedy added the comment:
I pushed Roger's patch so that the Windows section of config-keys.def is in a
consistent state. However, this does not fix the problem for
config-extensions.def or user versions of config-keys.def. A) The basic method
of the key dialog does not allow ent
Terry J. Reedy added the comment:
Yes, the deprecation in 3.3 did not apply to 2.7.
--
versions: +Python 3.4, Python 3.5 -Python 3.2, Python 3.3
___
Python tracker
<http://bugs.python.org/issue13
Changes by Terry J. Reedy :
--
stage: test needed -> patch review
___
Python tracker
<http://bugs.python.org/issue13790>
___
___
Python-bugs-list mai
Terry J. Reedy added the comment:
I just discovered what I consider to be a bug in parenmatch. Consider
(3 +
4 - 1)
When I type the closing paren or put the cursor on the second line and hit ^0,
the highlight extends from ( to ), inclusive, as it should. If I put the cursor
on the first
Terry J. Reedy added the comment:
Attached is the 3.4 file I am ready to commit. Comments on Rietveld. I believe
the 3 missed line could be hit by adding a delay to a test, but this seems
pretty useless. The test file explicitly tests only two of the methods. Tests
of other methods could be
Terry J. Reedy added the comment:
I did the trivial part of a 2.7 backport of 2 name changes, but the real
problem is that 2.7 does not have unittest.mock (which is why I have not used
it until now). I removed one use with a dummy class, but am leaving the other
one in the last test to you
Changes by Terry J. Reedy :
--
stage: patch review -> needs patch
___
Python tracker
<http://bugs.python.org/issue21694>
___
___
Python-bugs-list mai
Changes by Terry J. Reedy :
--
nosy: +serhiy.storchaka -gpolo
___
Python tracker
<http://bugs.python.org/issue7057>
___
___
Python-bugs-list mailing list
Unsub
Changes by Terry J. Reedy :
--
stage: -> test needed
versions: +Python 3.5 -Python 3.3
___
Python tracker
<http://bugs.python.org/issue21684>
___
___
Python-
Terry J. Reedy added the comment:
Looks sensible to me. Except for being name-only, this duplicates the api of
enumerate.
--
nosy: +terry.reedy
___
Python tracker
<http://bugs.python.org/issue21
Changes by Terry J. Reedy :
--
assignee: -> terry.reedy
nosy: +terry.reedy
___
Python tracker
<http://bugs.python.org/issue20043>
___
___
Python-bugs-list mai
Terry J. Reedy added the comment:
I added a direct test import of _thread. Since test_multiprocessing is one file
on 2.7 and AMD Fedora without threads is green for 2.7, in spite of the import
of threading, I left 2.7 alone. If that changes, this can be reopened and
marked for 2.7
Changes by Terry J. Reedy :
--
assignee: docs@python -> terry.reedy
nosy: +terry.reedy
___
Python tracker
<http://bugs.python.org/issue21726>
___
___
Python-
Changes by Terry J. Reedy :
--
resolution: -> fixed
stage: -> resolved
status: open -> closed
versions: +Python 2.7, Python 3.4, Python 3.5
___
Python tracker
<http://bugs.python.or
Changes by Terry J. Reedy :
--
assignee: -> terry.reedy
nosy: +terry.reedy
___
Python tracker
<http://bugs.python.org/issue21730>
___
___
Python-bugs-list mai
Terry J. Reedy added the comment:
It is helpful to note that an issue such as this does not apply to 2.7, so no
one needs to check whether the omission is an oversight, and that you know that
the patch applies cleanly to both 3.4 and 3.5.
--
resolution: -> fixed
stage: patch rev
Terry J. Reedy added the comment:
You might add a brief description of why this is good idea, or link to such a
discussion.
I am +1 on the idea, but a pydev discussion might be needed.
How did you test the patch? I don't think that passing the same tests in
non-verbose mode is good e
Terry J. Reedy added the comment:
Raymond, I was planning to do this today along with other small patches
(already done). Just say so and I will take it.
--
___
Python tracker
<http://bugs.python.org/issue19
Terry J. Reedy added the comment:
Please do. #21686 is for hyperparser test, which is the next one I want to look
at.
--
___
Python tracker
<http://bugs.python.org/issue21
Terry J. Reedy added the comment:
There are 7 scattered conditionals which only evaluate as one of True or False
during tests and 5 lines not hit at all as a result. I am emailing instructions
to install and use coverage package.
--
___
Python
Terry J. Reedy added the comment:
The current behavior is documented, if not exactly 'chosen' as the best
possible. "return the indices of the opening bracket and the closing bracket
(or the end of line, whichever comes first".
As you note, the automatic matching when p
Terry J. Reedy added the comment:
I suspect that the new end_at_eol parameter should take care of calltips.
--
___
Python tracker
<http://bugs.python.org/issue21
New submission from Terry J. Reedy:
idlelib.HyperParser.Hyperparser has these lines
_whitespace_chars = " \t\n\\"
_id_chars = string.ascii_letters + string.digits + "_"
_id_first_chars = string.ascii_letters + "_"
used in _eat_identifier() and get_expre
Changes by Terry J. Reedy :
--
dependencies: +IDLE - Test hyperparser
stage: -> test needed
___
Python tracker
<http://bugs.python.org/issue21756>
___
___
Py
Terry J. Reedy added the comment:
Since ParenMatch and fixes* to HyperParser depend on this issue, I would like
to do this first. For HyperParser.py, I would like to first commit a clean-up
only patch. The test_hyperparser patch should then only add the 'if __name__'
clause at t
Terry J. Reedy added the comment:
#21686 adds the test file that a new test would go in.
--
dependencies: +IDLE - Test hyperparser
___
Python tracker
<http://bugs.python.org/issue21
Terry J. Reedy added the comment:
General rule: If the other failures happen without the patch, then go ahead.
This is, unfortunately, all too common. If they only happen with the patch
(extremely unlikely for this one), then try to find out why.
Yes, this change should have a NEWS item
Terry J. Reedy added the comment:
I meant what you said in your last paragraph -- not passing a value for the new
parameter would keep current behavior.
--
___
Python tracker
<http://bugs.python.org/issue21
Terry J. Reedy added the comment:
I checked for usage: _id(_first)_chars is only used in _eat_identifier, which
is used in one place in get_expression. That is called once each in
AutoComplete and CallTips. Both are seldom visible accept as requested (by
waiting, for calltips). Calltips is
Terry J. Reedy added the comment:
I have never actually used this feature, and I do not see it in Notepad++. I
worry that some people, especially beginners with no experience of this
'feature' would hate this, so it would have to be optional.
Do you literally mean 'parentheses&
Terry J. Reedy added the comment:
Mark, On Windows, 3.4 comes with tck/tk 8.6. I am guessing that you built not
on windows. In any case, tkinter has had several patches since you last posted,
not all backported to 2.7, so an updated report with 3.4 or 3.5 and 8.6 would
help delineate the
Terry J. Reedy added the comment:
There is no entry for 'code object' (unlike, for instance 'class object').
There is an entry for 'code' and a subentry for 'object' with four links.
The fourth link is to the short definition in the library m
Changes by Terry J. Reedy :
--
nosy: -terry.reedy
___
Python tracker
<http://bugs.python.org/issue1102>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Terry J. Reedy :
--
resolution: -> fixed
stage: -> resolved
status: open -> closed
versions: +Python 2.7, Python 3.5
___
Python tracker
<http://bugs.python.or
Changes by Terry J. Reedy :
--
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue21559>
___
___
Python-bugs-list mailing list
Unsubscrib
Changes by Terry J. Reedy :
--
resolution: -> fixed
stage: -> resolved
___
Python tracker
<http://bugs.python.org/issue21559>
___
___
Python-bugs-list
Terry J. Reedy added the comment:
Coverage is now '99%'. Unless I see problems that are more than trivial, I am
going to polish the patch and commit.
I believe partial coverage of "for context in editwin.num_context_lines:" is
because the iterable is never empty. But
Terry J. Reedy added the comment:
Attached is the 3.4 patch as pushed. The main 'invisible' change, other than
the review comment, was to create the code string just once.
I discovered that the earlier version had a bug in saying 'ok' to the 3.x
HyperParser bug of no longe
Terry J. Reedy added the comment:
Idle Help, Completion sections "Two tabs in a row will supply the current ACW
selection, as will return or a double click." On 3.4 windows, tab-tab works,
one click dismisses the box immediately but without disabling it. Contrary to
the c
Changes by Terry J. Reedy :
--
resolution: -> not a bug
stage: -> resolved
status: open -> closed
___
Python tracker
<http://bugs.python.org/issue11394>
___
___
New submission from Terry J. Reedy:
3.0 introduced ... as Ellipsis literal. Test: add '...\n' to the end of the
test code. In test_get_expression, add at the end
p = get('12.3')
self.assertEqual(p.get_expression(), '...')
which
8901 - 9000 of 13135 matches
Mail list logo
