[issue21228] Missing enumeration of HTTPResponse Objects methods of urllib.request.urlopen's http.client.HTTPResponse?

2014-09-04 Thread Martin Panter
Martin Panter added the comment: With this patch, there is no longer any implication that the returned object implements the “addinfourl” interface. Perhaps that should be added back. Or maybe add it to the HTTPResponse class documentation itself? There is a comment that says the methods are t

[issue20912] [zipfile.py]: Make zip directory attributes more friendly for MS-Windows

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: If there are no objections I'll commit the patch soon. -- ___ Python tracker ___ ___ Python-bugs-l

[issue21951] tcl test change crashes AIX

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: The patch can be simpler. And there is yet one similar bug, for unicode strings. -- Added file: http://bugs.python.org/file36544/asobj_empty_sequence_2.patch ___ Python tracker _

[issue1508475] transparent gzip compression in urllib

2014-09-04 Thread Martin Panter
Martin Panter added the comment: Related: Issue 1243678, which includes a patch for “httplib” (now known as “http.client”?). That patch looks like it sets Accept-Encoding and decodes according to Content-Encoding. However I suspect it is also trying to be too “transparent” at the wrong level a

[issue1243678] httplib gzip support

2014-09-04 Thread Martin Panter
Martin Panter added the comment: Agreed, this issue is not a duplicate of the marked “gzip” seek issue, however it _does_ duplicate Issue 1508475. -- ___ Python tracker ___ __

[issue1152248] Add support for reading records with arbitrary separators to the standard IO stack

2014-09-04 Thread Martin Panter
Martin Panter added the comment: Related: * Issue 563491: 2002 proposal for parameter to readline, rejected at the time * Issue 17083: Newline is hard coded for binary file readline Fixing this issue for binary files would probably also satisfy Issue 17083. -- _

[issue17083] can't specify newline string for readline for binary files

2014-09-04 Thread Martin Panter
Changes by Martin Panter : -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue22253] ConfigParser does not handle files without sections

2014-09-04 Thread Martin Panter
Changes by Martin Panter : -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pytho

[issue19461] RawConfigParser modifies empty strings unconditionally

2014-09-04 Thread Łukasz Langa
Łukasz Langa added the comment: This bug has been fixed in Python 3.2 (see 892236137db9). Please use the configparser backport for Python 2.7. -- resolution: -> fixed stage: needs patch -> resolved status: open -> closed type: enhancement -> behavior __

[issue22339] Incorrect behavior when subclassing enum.Enum

2014-09-04 Thread Kiss György
Kiss György added the comment: Yes, sorry I forgot about that. Here is a minimal example: from enum import EnumMeta, Enum from types import DynamicClassAttribute class _MultiMeta(EnumMeta): def __init__(enum_class, cls, bases, classdict): # make sure we only have tuple values, not

[issue21933] Allow the user to change font sizes with the text pane of turtledemo

2014-09-04 Thread Terry J. Reedy
Changes by Terry J. Reedy : -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___ ___ Python-bugs

[issue21933] Allow the user to change font sizes with the text pane of turtledemo

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset ecc98ea50bc3 by Terry Jan Reedy in branch '3.4': Issue #21933: Make Control-Mousewhell only change font size and not also scroll. http://hg.python.org/cpython/rev/ecc98ea50bc3 -- ___ Python tracker

[issue22339] Incorrect behavior when subclassing enum.Enum

2014-09-04 Thread Ethan Furman
Ethan Furman added the comment: Can you give an example of the code you were having problems with? -- assignee: -> ethan.furman nosy: +ethan.furman ___ Python tracker ___ __

[issue22339] Incorrect behavior when subclassing enum.Enum

2014-09-04 Thread Ethan Furman
Changes by Ethan Furman : -- nosy: +barry, eli.bendersky ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:

[issue22339] Incorrect behavior when subclassing enum.Enum

2014-09-04 Thread Kiss György
New submission from Kiss György: There is a small inconvenience in the ``enum`` module. When I subclass ``enum.Enum`` and redefine the ``value`` dynamic attribute, the aliasing behavior doesn't work correctly, because ``member.value`` is used in some places instead of ``member._value_``. I atta

[issue22290] "AMD64 OpenIndiana 3.x" buildbot: assertion failed in PyObject_Call() in test_subprocess.test_preexec()

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: > For example, test_json crashed with SIGSEGV in this build, probably because > of an unhandled MemoryError exception: I created an issue for this bug: issue #22338. I proposed a fix. -- ___ Python tracker

[issue22338] test_json crash on memory allocation failure

2014-09-04 Thread STINNER Victor
Changes by STINNER Victor : -- type: -> crash versions: +Python 2.7, Python 3.4, Python 3.5 ___ Python tracker ___ ___ Python-bugs-li

[issue22338] test_json crash on memory allocation failure

2014-09-04 Thread STINNER Victor
New submission from STINNER Victor: Using pyfailmalloc, I'm able to reproduce the crash seen on a buildbot. Attached patch fixes two bugs in error handlers. http://buildbot.python.org/all/builders/AMD64%20OpenIndiana%203.x/builds/8557/steps/test/logs/stdio [191/390] test_urllib2net Fatal Pytho

[issue22290] "AMD64 OpenIndiana 3.x" buildbot: assertion failed in PyObject_Call() in test_subprocess.test_preexec()

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 16e3d240456f by Victor Stinner in branch 'default': Issue #22290: PyObject_Call() now fails with an assertion error when called http://hg.python.org/cpython/rev/16e3d240456f -- nosy: +python-dev ___ Pytho

[issue22290] "AMD64 OpenIndiana 3.x" buildbot: assertion failed in PyObject_Call() in test_subprocess.test_preexec()

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: I reproduced the crash with pyfailmalloc. The problem is that _posixsubprocess.fork_exec() calls gc.enable() on error when preexec_fn is set and the garbage collector was disabled (by fork_exec). Calling a function with an exception set is forbidden. Attached

[issue21907] Update Windows build batch scripts

2014-09-04 Thread David Bolen
David Bolen added the comment: While troubleshooting an issue with test_distutils consistently failing on my XP buildbot, I narrowed it down to the test.bat change to use run_tests.py. I don't yet know fully what's happening, but after replacing the new test.bat with the older version (that c

[issue4130] Intel icc 9.1 does not support __int128_t used by ctypes

2014-09-04 Thread Marcin Wojdyr
Marcin Wojdyr added the comment: I just came across this issue when compiling 2.7.8 with ICC 15.0. But apparently it's been fixed in the meantime by updating libffi to v3.1 (r92061 for 2.7). -- nosy: +wojdyr ___ Python tracker

[issue21228] Missing enumeration of HTTPResponse Objects methods of urllib.request.urlopen's http.client.HTTPResponse?

2014-09-04 Thread Evens Fortuné
Evens Fortuné added the comment: Hi, >From my limited experience reporting documentation issues, I see that it's >better to submit a patch than to only report an issue. So, I've tried to look >into the source code to figure out what was going on. I have attached a patch >that I'm submitting t

[issue22117] Rewrite pytime.h to work on nanoseconds

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: And now, something completly different. nanosec-wip.patch is a work-in-progress patch to use a unified (64 bits) integer type to store a timestamp: _PyTime_t. The type is written to be opaque, the unit is undefined, you must use functions to convert from and t

[issue22337] Modulus not returning proper remainder

2014-09-04 Thread R. David Murray
R. David Murray added the comment: This is, indeed, the way Python works. See https://docs.python.org/3/reference/expressions.html where you will find: The modulo operator always yields a result with the same sign as its second operand (or zero); the absolute value of the result is strictly

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: Debug trace on FreeBSD 7.2: == FAIL: test_splitlist (test.test_tcl.TclTest) -- Traceback (most recent call last): File "/us

[issue22337] Modulus not returning proper remainder

2014-09-04 Thread DenisCrnic
New submission from DenisCrnic: Modulus is returning wrong remainder when working with negative numbers. For example 34%-3 should return 1, but it returns -2. 34/-3=-11, remainder: 1, proof: -11*(-3)+1=34 <--- Math logic (works in Java for example) but python goes like that: 34/-3=12, remainde

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: Debug trace from x86 FreeBSD 6.4 3.x: == FAIL: test_splitlist (test.test_tcl.TclTest) -- Traceback (most recent call last):

[issue20421] expose SSL socket protocol version

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 16c86a6bdbe2 by Alex Gaynor in branch '2.7': Issue #20421: Add a .version() method to SSL sockets exposing the actual protocol version in use. http://hg.python.org/cpython/rev/16c86a6bdbe2 -- ___ Python

[issue4180] warnings.simplefilter("always") does not make warnings always show up

2014-09-04 Thread Antoine Pitrou
Antoine Pitrou added the comment: Brett, do you want to review this? -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubs

[issue20421] expose SSL socket protocol version

2014-09-04 Thread Alex Gaynor
Alex Gaynor added the comment: Attached patch backports it (only change is the use of `closing()` and resolving the conflict in Misc/NEWS). I'll leave it up to benjamin whether he wants to commit (input from others welcome). My view is to prefer backporting stuff since it helps keep the diff s

[issue20421] expose SSL socket protocol version

2014-09-04 Thread Antoine Pitrou
Antoine Pitrou added the comment: It's as you want, now. I don't think this is really important, though. -- ___ Python tracker ___ ___

[issue20421] expose SSL socket protocol version

2014-09-04 Thread Alex Gaynor
Alex Gaynor added the comment: Should this be backported to 2.7.9? -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubsc

[issue21951] tcl test change crashes AIX

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: Thanks David! Your ouput confirms my analysis. asobj_empty_sequence.patch should fix the test on AIX. I read Tcl source code, Tcl_NewListObj() doesn't use the second parameter if the first parameter (length) is zero. -- Added file: http://bugs.python.

[issue13179] IDLE uses common tkinter variables across all editor windows

2014-09-04 Thread Terry J. Reedy
Terry J. Reedy added the comment: There has been more discussion of code context toggling in #17535, msg225411 2014-8-16 and following messages. The code context button toggles both the state of the current window and the default for new windows. PEP 434 ratified my comment about new Idle feat

[issue20421] expose SSL socket protocol version

2014-09-04 Thread Antoine Pitrou
Antoine Pitrou added the comment: Pushed to default. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ __

[issue17535] IDLE: Add an option to show line numbers along the left side of the editor window, and have it enabled by default.

2014-09-04 Thread Terry J. Reedy
Terry J. Reedy added the comment: And if you enable code context in the other window, neither will have the checkmark. As I said, the one button is being used to toggle two things: the state of the current window and the default for future windows, and the checkmark only indicates the second.

[issue20421] expose SSL socket protocol version

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 648685f8d5e9 by Antoine Pitrou in branch 'default': Issue #20421: Add a .version() method to SSL sockets exposing the actual protocol version in use. http://hg.python.org/cpython/rev/648685f8d5e9 -- nosy: +python-dev __

[issue21933] Allow the user to change font sizes with the text pane of turtledemo

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: This works on X11 (Linux). Actually it doesn't matter for what widget you call bind_all(). It works globally. -- ___ Python tracker ___

[issue21933] Allow the user to change font sizes with the text pane of turtledemo

2014-09-04 Thread Terry J. Reedy
Terry J. Reedy added the comment: Great. That works better on Windows too. Attached is an augmented patch to also move the other size bindings to text (I verified that 'bind_all' is needed), move the resize binding back to where it was, delete the now-empty binding function, and add a not to t

[issue21951] tcl test change crashes AIX

2014-09-04 Thread David Edelsohn
David Edelsohn added the comment: $ ./python -m test -v test_tcl == CPython 3.5.0a0 (default:9ab404cdcaa1+, Sep 4 2014, 10:06:33) [GCC 4.8.1] == AIX-1-00F84C0C4C00-powerpc-32bit big-endian == hash algorithm: siphash24 32bit == /home/dje/src/cpython/build/test_python_48694044 Testing with fl

[issue22336] _tkinter should use Python PyMem_Malloc() instead of Tcl ckalloc()

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Event structs (Tkapp_CallEvent, VarEvent, CommandEvent) must have been allocated by the caller using Tcl_Alloc or ckalloc (see man Tcl_ThreadQueueEvent). -- ___ Python tracker

[issue22336] _tkinter should use Python PyMem_Malloc() instead of Tcl ckalloc()

2014-09-04 Thread STINNER Victor
Changes by STINNER Victor : -- components: +Tkinter ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[issue22336] _tkinter should use Python PyMem_Malloc() instead of Tcl ckalloc()

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: >PyMem_Free(FREECAST argv); FREECAST can be dropped here, PyMem_Free() always takes a void* pointer. -- ___ Python tracker ___

[issue22336] _tkinter should use Python PyMem_Malloc() instead of Tcl ckalloc()

2014-09-04 Thread STINNER Victor
New submission from STINNER Victor: The PyMem_Malloc(size) function has a well defined behaviour: if size is 0, a pointer different than NULL is returned. It looks like ckalloc(size) returns NULL if the size is NULL: see issue #21951 (bug on AIX). Moreover, memory allocated by ckalloc() is not

[issue21951] tcl test change crashes AIX

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: > When the failing test is re-enabled, the _tkinter patch produces the > following output: Thanks. I found and fixed a bug in AsObj() for tuple and list. It looks like ckalloc(size) returns NULL if size is 0. Please update Python, compile and run test_tcl to

[issue21951] tcl test change crashes AIX

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 9ab404cdcaa1 by Victor Stinner in branch 'default': Issue #21951: Fix AsObj() of the _tkinter module: raise MemoryError on memory http://hg.python.org/cpython/rev/9ab404cdcaa1 -- ___ Python tracker

[issue21951] tcl test change crashes AIX

2014-09-04 Thread David Edelsohn
David Edelsohn added the comment: When the failing test is re-enabled, the _tkinter patch produces the following output: test_user_command (test.test_tcl.TclTest) ... Assertion failed: __EX, file /home/dje/src/cpython/Modules/_tkinter.c, line 1277 Fatal Python error: Aborted Current thread 0

[issue21933] Allow the user to change font sizes with the text pane of turtledemo

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Here is a patch which should fix this bug. In additional it fixes a bug on MacOS: mouse wheel only increased font size. -- Added file: http://bugs.python.org/file36532/turtledemo_change_font_size.patch ___ Python t

[issue21951] tcl test change crashes AIX

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: As far as I remember, both res and PyErr_Occurred() were NULL. -- ___ Python tracker ___ ___ Pytho

[issue22335] Python 3: Segfault instead of MemoryError when bytearray too big

2014-09-04 Thread STINNER Victor
Changes by STINNER Victor : -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue22335] Python 3: Segfault instead of MemoryError when bytearray too big

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Looks as integer overflow introduced in issue19087. +if (size + logical_offset + 1 < alloc) { -- nosy: +pitrou, serhiy.storchaka stage: -> needs patch versions: +Python 3.5 ___ Python tracker

[issue22323] Rewrite PyUnicode_AsWideChar() and PyUnicode_AsWideCharString()

2014-09-04 Thread STINNER Victor
Changes by STINNER Victor : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue21356] Support LibreSSL (instead of OpenSSL): make RAND_egd optional

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: Related discussion: http://marc.info/?l=openbsd-tech&m=140512043210089&w=2 The answer for Python is: "your package maintainers and ask them to configure these software without egd support." -- ___ Python tracker

[issue22335] Python 3: Segfault instead of MemoryError when bytearray too big

2014-09-04 Thread swanson
New submission from swanson: On Python 3, but not Python 2, you crash with a Segmentation Fault instead of getting a MemoryError as expected. It seems to only be a problem with bytearray, not with other things like tuple: $ python3 Python 3.4.0 (default, Apr 11 2014, 13:05:18) [GCC 4.8.2] on

[issue21356] Support LibreSSL (instead of OpenSSL): make RAND_egd optional

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: > The PyPy patch (and some discussion) is here: Your patch checks at runtime if libssl comes with RAND_egd: HAVE_OPENSSL_RAND_EGD = rffi_platform.Has('RAND_egd') In CPython, the _ssl module is compiled in C. How can we check if libssl provides RAND_egd() o

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: > More likely there is a bug in Tcl 8.6b1 (as in Tcl before 8.5.5). If the bug is not present in Tcl 8.6.2, we can skip the test on the version Tcl 8.6b1. -- ___ Python tracker _

[issue21186] RawConfigParser __name__ option handling inconsistent

2014-09-04 Thread Łukasz Langa
Łukasz Langa added the comment: Thanks for your report and sorry it took so long to respond! You are absolutely right that __name__ was a broken feature, which is why we removed it in Python 3.2. See issue #10489. For Python 2.7 you can use the backport, available on PyPI. -- resoluti

[issue19546] configparser leaks implementation detail

2014-09-04 Thread Claudiu Popa
Claudiu Popa added the comment: No problem, thank you for committing it. -- ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue19546] configparser leaks implementation detail

2014-09-04 Thread Łukasz Langa
Łukasz Langa added the comment: Thank you Claudiu for your patch, sorry it took so long to respond. -- ___ Python tracker ___ ___ Pyth

[issue19546] configparser leaks implementation detail

2014-09-04 Thread Łukasz Langa
Changes by Łukasz Langa : -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___ ___ Python-bugs-l

[issue19546] configparser leaks implementation detail

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 2b14665b7bce by Łukasz Langa in branch 'default': Fix #19546: onfigparser exceptions expose implementation details. Patch by Claudiu Popa. http://hg.python.org/cpython/rev/2b14665b7bce New changeset 554ead559f24 by Łukasz Langa in branch 'default'

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread Ned Deily
Ned Deily added the comment: Perhaps David B can update Tcl and Tk on the buildbot? Note, Tcl/Tk 8.6.2 (and 8.5.16) were just released a few days ago. -- nosy: +db3l, ned.deily ___ Python tracker _

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: More likely there is a bug in Tcl 8.6b1 (as in Tcl before 8.5.5). -- ___ Python tracker ___ ___ Py

[issue21933] Allow the user to change font sizes with the text pane of turtledemo

2014-09-04 Thread Terry J. Reedy
Terry J. Reedy added the comment: I am aware of this because the Windows has the same behavior. As I noted in my post to #17642, I consider this behavior a tolerable glitch rather than a patch-blocking bug for turtledemo because the text is relatively short and read only, so there is no issue

[issue6639] turtle: _tkinter.TclError: invalid command name ".10170160"

2014-09-04 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Here is more complete patch. -- Added file: http://bugs.python.org/file36531/turtledemo_destroy.patch ___ Python tracker ___ __

[issue21951] tcl test change crashes AIX

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: The failing assertion means that the method raised an exception and returned a result, or failed but didn't raised an exception. According to the gdb trace, the function failed (res is NULL). You can display the current exception in gdb using: "print PyErr_Occ

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset e6cfbc3e2ed9 by Victor Stinner in branch 'default': Issue #22334: Add debug traces to test_tcl http://hg.python.org/cpython/rev/e6cfbc3e2ed9 -- nosy: +python-dev ___ Python tracker

[issue22334] test_tcl.test_split() fails on "x86 FreeBSD 7.2 3.x" buildbot

2014-09-04 Thread STINNER Victor
New submission from STINNER Victor: The test was added by the issue #18101 (changeset 9486c07929a1). On FreeBSD 7.2, Tcl version is 8.6b1 (seen in the test output). http://buildbot.python.org/all/builders/x86%20FreeBSD%207.2%203.x/builds/5560/steps/test/logs/stdio =

[issue22333] test_threaded_import.test_parallel_meta_path() failed on x86 Windows7 3.x

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset f8d2834e1269 by Victor Stinner in branch 'default': Issue #22333: Add debug traces to test_threaded_import http://hg.python.org/cpython/rev/f8d2834e1269 -- nosy: +python-dev ___ Python tracker

[issue22333] test_threaded_import.test_parallel_meta_path() failed on x86 Windows7 3.x

2014-09-04 Thread STINNER Victor
New submission from STINNER Victor: == FAIL: test_parallel_meta_path (test.test_threaded_import.ThreadedImportTests) -- Traceback (most recent call last): File

[issue22258] set_inheritable(): ioctl(FIOCLEX) is available but fails with ENOTTY on Illumos

2014-09-04 Thread STINNER Victor
STINNER Victor added the comment: > "the the" Oh, thanks for the report. It's now fixed. It would be better to have the Misc/NEWS entry in the patch directly :-( -- ___ Python tracker

[issue22258] set_inheritable(): ioctl(FIOCLEX) is available but fails with ENOTTY on Illumos

2014-09-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset 9d5386a22e68 by Victor Stinner in branch 'default': Issue #22258: Fix typo in Misc/NEWS http://hg.python.org/cpython/rev/9d5386a22e68 -- ___ Python tracker ___

[issue19690] test_logging test_race failed with PermissionError

2014-09-04 Thread Vinay Sajip
Vinay Sajip added the comment: > What is the status of this issue? Sorry, it's been off my radar - I wonder if it has become more common recently. It's hard to reproduce, and tweaking timeouts might make it disappear, but it's a trial-and-error situation because I can hardly ever get the error