[issue28976] incorrect description that dose not conform to the actual behavior

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

Can you provide a code snippet that demonstrates the actual problem? As far as 
I can see this code behaves as documented

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue28976>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28976] incorrect description that dose not conform to the actual behavior

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

How does that contradict the documentation? Both print statements were executed 
- the second one raised an exception because of 'op' not being defined.

--

___
Python tracker 
<http://bugs.python.org/issue28976>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28976] incorrect description that dose not conform to the actual behavior

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

Indeed, but that's merely because an exception has been raised. The example 
given doesn't include any statements that would raise an exception.

I'm not sure it's necessary to re-iterate well known Python behaviour in this 
section.

Anyway, that's my 2c

--

___
Python tracker 
<http://bugs.python.org/issue28976>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28979] What's New entry on compact dict mentions "faster" implementation

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

Agree. Patch provided.

--
keywords: +patch
nosy: +brendan-donegan
Added file: http://bugs.python.org/file45919/whatsnew36.patch

___
Python tracker 
<http://bugs.python.org/issue28979>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28976] incorrect description that dose not conform to the actual behavior

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

None represents the case where the if statement is false

On Thu, 15 Dec 2016 at 20:40 woo yoo  wrote:

>
> woo yoo added the comment:
>
> According to the original description, "either all or none...",what does
> 'none' represent?
>
> --
>
> ___
> Python tracker 
> <http://bugs.python.org/issue28976>
> ___
>

--
nosy: +brendand

___
Python tracker 
<http://bugs.python.org/issue28976>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28976] incorrect description that dose not conform to the actual behavior

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

I think the consensus is that the wording is correct. If you can come up
with text that is clearer, and still correct, please do submit a patch.

On Thu, 15 Dec 2016 at 20:50 woo yoo  wrote:

>
> woo yoo added the comment:
>
> "either all or none of the print() calls are executed",I think the 'none'
> describes the number of the calls to be executed.
>
> --
>
> ___
> Python tracker 
> <http://bugs.python.org/issue28976>
> ___
>

--

___
Python tracker 
<http://bugs.python.org/issue28976>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue27069] webbrowser creates zombi processes in the background mode

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

This appears to be Firefox specific? At least:

b = webbrowser.get("chromium-browser")
b.open("https://bugs.python.org/issue26741";)

returns simply:

True

With no exception

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue27069>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28980] ResourceWarning when imorting antigravity in 3.6

2016-12-15 Thread Brendan Donegan

Brendan Donegan added the comment:

I think this is essentially a duplicate of http://bugs.python.org/issue27069, 
or at least fixing that will fix this (as the exception is caused by the 
webbrowser.open() call)

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue28980>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue28989] .dll files missing

2016-12-16 Thread Brendan Donegan

Brendan Donegan added the comment:

This more likely to be a system misconfiguration issue than a Python one. 
Following 
http://stackoverflow.com/questions/33265663/api-ms-win-crt-runtime-l1-1-0-dll-is-missing-when-opening-microsoft-office-file
 will probably fix the issue for you.

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue28989>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29010] Incorrect description about scope related with inheritance

2016-12-19 Thread Brendan Donegan

Brendan Donegan added the comment:

Hi. Can you please provide a small code snippet illustrating where this 
diverges from the actual behaviour?

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue29010>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29010] Incorrect description about scope related with inheritance

2016-12-19 Thread Brendan Donegan

Brendan Donegan added the comment:

IMO there's no actual bug in Python. Perhaps documentation can be clarified
in a short simple way. I would reiterate the point that restating facts of
the language all over the documentation is not a great idea, but maybe here
the benefits outweigh the drawbacks.

Anyway I think the bug is clear now (it's a docs issue if anything) so I'd
leave it open until someone offers a patch or a strong argument why it
shouldn't be clarified.

On Tue, 20 Dec 2016 at 09:18 woo yoo  wrote:

>
> woo yoo added the comment:
>
> If i was wrong about this issue, please tell me.
>
> --
>
> ___
> Python tracker 
> <http://bugs.python.org/issue29010>
> ___
>

--
nosy: +brendand

___
Python tracker 
<http://bugs.python.org/issue29010>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29010] Incorrect description about scope related with inheritance

2016-12-20 Thread Brendan Donegan

Brendan Donegan added the comment:

Worth putting them side by side I think:

"The name BaseClassName must be defined in a scope containing the derived class 
definition"

vs.

"BaseClassName must be defined in a namespace accessible from the scope 
containing the derived class definition"

--

___
Python tracker 
<http://bugs.python.org/issue29010>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29061] secrets.randbelow(-1) hangs

2016-12-24 Thread Brendan Donegan

Brendan Donegan added the comment:

Reproducible on Linux as well, I think I know where the issue is and will try 
to submit a patch soon.

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue29061>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29061] secrets.randbelow(-1) hangs

2016-12-24 Thread Brendan Donegan

Brendan Donegan added the comment:

Ok, here it is. My first code patch in Python. 

Basically the existing code was depending on bit_length to DTRT and raise a 
ValueError, but negative numbers have a positive bit length. Then when it hits:

234 while r >= n:   
 
235 r = getrandbits(k)  

It just spins on that as r is always going to be greater than a negative number.

I tried not to be too clever so just put a guard early in the function. This 
has the added advantage of giving us a clearer error message.

--
keywords: +patch
Added file: http://bugs.python.org/file46021/issue_29061_randbelow.patch

___
Python tracker 
<http://bugs.python.org/issue29061>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29061] secrets.randbelow(-1) hangs

2016-12-27 Thread Brendan Donegan

Brendan Donegan added the comment:

Hi Raymond,

I have done that when creating the patch and have confirmation in my inbox
- perhaps Ewa hasn't filed it yet?

On Tue, 27 Dec 2016 at 14:43 Raymond Hettinger 
wrote:

>
> Raymond Hettinger added the comment:
>
> Brendan, would you please submit a contributor agreement.
>
> --
> priority: high -> normal
>
> ___
> Python tracker 
> <http://bugs.python.org/issue29061>
> ___
>

--
nosy: +brendand

___
Python tracker 
<http://bugs.python.org/issue29061>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29061] secrets.randbelow(-1) hangs

2016-12-27 Thread Brendan Donegan

Brendan Donegan added the comment:

Ok, here's a second version of the patch. Normally I don't like testing 
multiple things in one test but I've gone with what seems to be the convention 
here in test_secrets.py

--
Added file: http://bugs.python.org/file46054/issue_29061_randbelow_v2.patch

___
Python tracker 
<http://bugs.python.org/issue29061>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29061] secrets.randbelow(-1) hangs

2016-12-27 Thread Brendan Donegan

Brendan Donegan added the comment:

If I'm not mistaken, _randbelow is defined in Random, which SystemRandom
inherits from. Just for clarity

On Tue, 27 Dec 2016 at 22:08 Josh Rosenberg  wrote:

>
> Josh Rosenberg added the comment:
>
> SystemRandom._randbelow has this problem, perhaps it should be fixed
> there, not in one of many possible wrappers for it?
>
> --
> nosy: +josh.r
>
> ___
> Python tracker 
> <http://bugs.python.org/issue29061>
> ___
>

--

___
Python tracker 
<http://bugs.python.org/issue29061>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue29276] HTMLParser in Python 2.7 doesn't recognize image tags wrapped up in link tags

2017-01-14 Thread Brendan Donegan

Brendan Donegan added the comment:

I even get the correct behaviour in 2.7.12:
Python 2.7.12+ (default, Sep 17 2016, 12:08:02) 
[GCC 6.2.0 20160914] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from HTMLParser import HTMLParser
>>> class MyHTMLParser(HTMLParser):
... def handle_starttag(self, tag, attrs):
... print("Encountered a start tag: %s" % tag)
... 
>>> parser = MyHTMLParser()
>>> parser.feed('http://somesite.com/large_image.jpg";>>> src="http://somesite.com/small_image.jpg"; width="800px" />')
Encountered a start tag: a
Encountered a start tag: img

Ari, can you provide more info about the exact version and platform you are 
using?

--
nosy: +brendan-donegan

___
Python tracker 
<http://bugs.python.org/issue29276>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com