[issue47198] os.stat on windows doesn't take an open file even though os.stat in os.supports_fd

2022-04-01 Thread Joe Cool
New submission from Joe Cool : os.stat on windows doesn't take an open file even though os.stat in os.supports_fd >>> fd = open('tmp.tmp', 'w') >>> fd <_io.TextIOWrapper name='tmp.tmp' mode='w' encoding='cp1252

[issue46947] unicodedata.name gives ValueError for control characters

2022-03-07 Thread Joe Cool
Joe Cool added the comment: My recommendation would be to add a keyword parameter, defaulting to False, to name(), something like give_full_alias, or maybe errors=“give_full_alias” like the IO functions. In the meantime, as the author of perllib, I had to make my own dict to return to the

[issue46947] unicodedata.name gives ValueError for control characters

2022-03-07 Thread Joe Cool
Joe Cool added the comment: Note: This is an issue for all chars in the ordinal range 0 thru 31. -- ___ Python tracker <https://bugs.python.org/issue46

[issue46947] unicodedata.name gives ValueError for control characters

2022-03-07 Thread Joe Cool
New submission from Joe Cool : unicodedata.name gives ValueError for control characters, for example: >>> unicodedata.name('\x00') Traceback (most recent call last): File "", line 1, in ValueError: no such name >>> unicodedata.name('\t') Trac

[issue46931] zipfile library will raise uncaught oserror when reading length incorrect zip file

2022-03-05 Thread Vertu Joe
New submission from Vertu Joe : I intentionally made some corrupted zip archive files for testing. If some contents were removed from the archive instead of changing the bits. when trying to read such files, the zipfile will raise an uncaught OSError, instead of a badzipfile error as

[issue40467] subprocess: replacement shell on windows with executable="..." arg

2021-12-03 Thread Joe Cool
Joe Cool added the comment: Proposed solution: if comspec.endswith('sh.exe') or comspec.endswith('sh'):# issue 40467 args = '{} -c "{}"'.format (comspec, args)

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-11-09 Thread Joe Wells
Joe Wells added the comment: Some quick comments on Martin's pull request. 1. The changes are sufficient to let the user make things work the way it is requested that they work and anyone who does not start using the new format_locals parameter will get the old behavior. 2. A side co

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-10-28 Thread Joe Wells
Joe Wells added the comment: 1. As background, it is worth remembering that a major motivation for why FrameSummary.__init__ stringifies the local variable values in its parameter locals is to prevent the resulting data structure from keeping those values live. This enables them to be

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-10-20 Thread Joe Wells
Joe Wells added the comment: In the hopes of convincing someone to install a fix to this bug, I will mention a few additional points. When I mention “the capture_locals feature”, I mean calls of the form traceback.TracebackException(..., capture_locals=True) and

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-10-20 Thread Joe Wells
Joe Wells added the comment: Andrei, thanks very much for the pointer to bug/issue https://bugs.python.org/issue39228. I had not noticed the earlier comment by Irit pointing to that bug. (Is there some way to merge bugs so that someone visiting one of the bugs will see the merged stream

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-10-20 Thread Joe Wells
Joe Wells added the comment: I'm sorry Andrei: I misread your alteration of my example and misunderstood its purpose. For anyone else reading these messages: In my most recent comment above, please ignore the part of my comment about Andrei's example. So yes, Andrei, that is

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-10-20 Thread Joe Wells
Joe Wells added the comment: Here are my thoughts inspired by Andrei's insightful comments. 1. In response to the major issue Andrei raises, I agree that it is desirable that repr would never raise an exception. The reasons Andrei mentions seem quite correct to me. However, I thin

[issue43656] TracebackException or StackSummary.extract with capture_locals=True fail to catch exceptions raised by repr() on value of frame local variable in FrameSummary.__init__.

2021-10-19 Thread Joe Wells
Joe Wells added the comment: I would like to request that this bug be repopened and fixed. I've changed (or at least tried to change, I'm not sure if it will let me) the title of the bug to point out that the failure happens in FrameSummary.__init__. It does not

[issue39232] asyncio crashes when tearing down the proactor event loop

2021-06-22 Thread Joe
Joe added the comment: We are running into this all the time, ever since the Proactor became the default on Windows in 3.8. Usually it comes up when the program terminates due to an unhandled exception during a highly concurrent operation. The resulting cascade of RuntimeErrors often

[issue44114] Incorrect function signatures in dictobject.c

2021-05-12 Thread Joe Marshall
Change by Joe Marshall : -- keywords: +patch pull_requests: +24703 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26062 ___ Python tracker <https://bugs.python.org/issu

[issue44114] Incorrect function signatures in dictobject.c

2021-05-12 Thread Joe Marshall
New submission from Joe Marshall : There's a couple of wrong function signatures in dictobject.c, dictkeys_reversed and dictitems_reversed are defined as single arg functions like so: PyObject *(PyObject *), and are then passed around and called as PyCFunctions, which should be PyO

[issue39661] TimedRotatingFileHandler doesn’t handle DST switch with daily rollover

2020-02-17 Thread Joe Cool
Joe Cool added the comment: Never mind. I was looking for the DST code in computeRollover, and I found it in doRollover. -- stage: -> resolved status: open -> closed ___ Python tracker <https://bugs.python.org/i

[issue39661] TimedRotatingFileHandler doesn’t handle DST switch with daily rollover

2020-02-17 Thread Joe Cool
New submission from Joe Cool : TimedRotatingFileHandler doesn’t handle the switch to/from DST when using daily/midnight rotation. It does not adjust the rollover time so the rollover will be off by an hour. Parameters: when=‘midnight’, utc=False -- components: Library (Lib) messages

[issue39196] json fails to encode dictionary view types

2020-01-02 Thread Joe Gordon
New submission from Joe Gordon : Python 3 fails to encode dictionary view objects. Assuming this is an expected behavior, what is the thinking behind it? I was unable to find any documentation around this. > import json; json.dumps({}.values()) "TypeError: Object of type dict_value

[issue37270] Manage memory lifetime for all type-related objects.

2019-06-13 Thread Joe Jevnik
Change by Joe Jevnik : -- keywords: +patch pull_requests: +13925 stage: -> patch review pull_request: https://github.com/python/cpython/pull/14066 ___ Python tracker <https://bugs.python.org/issu

[issue37270] Manage memory lifetime for all type-related objects.

2019-06-13 Thread Joe Jevnik
New submission from Joe Jevnik : When using PyType_FromSpec, the memory for PyType_Spec.name, Py_tp_methods, and Py_tp_members needs to somehow outlive the resulting type. This makes it hard to use this interface to generate types without just leaking the memory for these arrays, which is

[issue36760] subprocess.run fails with capture_output=True and stderr=STDOUT

2019-04-30 Thread Joe Borg
New submission from Joe Borg : Reading from https://docs.python.org/3/library/subprocess.html#subprocess.CompletedProcess """ If you ran the process with stderr=subprocess.STDOUT, stdout and stderr will be combined in this attribute, and stderr will be None. """

[issue36068] Make _tuplegetter objects serializable

2019-02-21 Thread Joe Jevnik
Joe Jevnik added the comment: Thank you for reviewing this so quickly! -- ___ Python tracker <https://bugs.python.org/issue36068> ___ ___ Python-bugs-list mailin

[issue36068] Make _tuplegetter objects serializable

2019-02-21 Thread Joe Jevnik
Change by Joe Jevnik : -- keywords: +patch pull_requests: +12006 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue36068> ___ ___ Py

[issue36068] Make _tuplegetter objects serializable

2019-02-21 Thread Joe Jevnik
New submission from Joe Jevnik : The new _tuplegetter objects for accessing fields of a namedtuple are no longer serializable with pickle. Cloudpickle, a library which provides extensions to pickle to facilitate distributed computing in Python, depended on being able to pickle the members of

[issue34944] Update _dirnameW to accept long path names

2018-10-09 Thread Joe Pamer
Change by Joe Pamer : -- keywords: +patch pull_requests: +9157 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue34944> ___ ___ Python-

[issue34944] Update _dirnameW to accept long path names

2018-10-09 Thread Joe Pamer
New submission from Joe Pamer : The fix for issue 32557 updated os__getdiskusage_impl to use _dirnameW for obtaining the parent directory of a file. This would cause a regression if the path exceeded 260 characters, since _dirnameW currently returns -1 if given a path >= MAX_PATH in len

[issue32557] allow shutil.disk_usage to take a file path on Windows also

2018-09-20 Thread Joe Pamer
Joe Pamer added the comment: Awesome - thanks, Steve - this is all super helpful! If you're cool with it I'd like to stick to using _dirnameW for now, and then follow up with another set of PRs for the fixes you've recommended. -- ___

[issue32557] allow shutil.disk_usage to take a file path on Windows also

2018-09-19 Thread Joe Pamer
Joe Pamer added the comment: Just to loop back, I updated the PR to avoid MAX_PATH and only allocate in the "not a directory" case. Thanks for getting back to me so quickly! One question, though, is that it *does* seem like MAX_PATH is still referenced in several places in pos

[issue32557] allow shutil.disk_usage to take a file path on Windows also

2018-09-18 Thread Joe Pamer
Joe Pamer added the comment: Got it - thanks! That sounds good to me, so I'll take a look at how other functions are working around MAX_PATH and update the diff to also avoid the copy when possible. -- ___ Python tracker <https://bugs.py

[issue32557] allow shutil.disk_usage to take a file path on Windows also

2018-09-17 Thread Joe Pamer
Joe Pamer added the comment: Hi! I decided to try fixing this one as a way to get acquainted with the code base. I went ahead and updated the backing NT C function, but please let me know if you'd prefer I update disk_usage as proposed. Thanks! -- nosy: +jo

[issue23927] getargs.c skipitem() doesn't skip 'w*'

2018-07-23 Thread Joe Jevnik
Change by Joe Jevnik : -- pull_requests: +7946 ___ Python tracker <https://bugs.python.org/issue23927> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34078] Broken CRL functionality in ssl.py

2018-07-09 Thread Joe N
New submission from Joe N : CRLs in ssl.py or at the documentation is broken. Specifically I think the documentation here is wrong: https://docs.python.org/3/library/ssl.html#ssl.SSLContext.load_verify_locations Here is a stackoverflow post: https://stackoverflow.com/questions/51196492/how

[issue23926] skipitem() in getargs.c still supports 'w' and 'w#', and shouldn't

2018-07-09 Thread Joe Jevnik
Change by Joe Jevnik : -- pull_requests: +7754 ___ Python tracker <https://bugs.python.org/issue23926> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue31951] import curses is broken on windows

2017-11-07 Thread joe m
joe m added the comment: I would much prefer the curses module to be supported in newer versions since I believe that curses is installed as a built in module (not sure about that). Anyhow, thank you for your help but I have found a replacement module called "asciimatics" which

[issue31951] import curses is broken on windows

2017-11-05 Thread joe m
New submission from joe m : Importing curses on Windows install calls the following: Traceback (most recent call last): File "", line 1, in File "C:\Users\user name\AppData\Local\Programs\Python\Python36-32\lib\curses\__init__.py", line 13, in

[issue31557] tarfile: incorrectly treats regular file as directory

2017-10-04 Thread Joe Tsai
Joe Tsai added the comment: It creates a number of nested directories only because GNU (and BSD) tar implicitly create missing parent directories. If you cd into the bottom-most folder, you will see "foo.txt". -- ___ Python track

[issue31557] tarfile: incorrectly treats regular file as directory

2017-10-03 Thread Joe Tsai
Joe Tsai added the comment: This bug is not platform specific. I've attached a reproduction: $ python >>> import tarfile >>> tarfile.open("test.tar", "r").next().isdir() True $ tar -tvf test.tar -rw-rw-r-- 0/0 0 1969-12-31 16:00 123456

[issue31557] tarfile: incorrectly treats regular file as directory

2017-09-22 Thread Joe Tsai
New submission from Joe Tsai: The original V7 header only allocates 100B to store the file path. If a path exceeds this length, then either the PAX format or GNU formats must be used, which can represent arbitrarily long file paths. When doing so, most tar writers just store the first 100B of

[issue28638] Optimize namedtuple creation

2017-07-18 Thread Joe Jevnik
Joe Jevnik added the comment: I added a benchmark suite (using Victor's perf utility) to cnamedtuple. The results are here: https://github.com/ll/cnamedtuple#benchmarks To summarize: type creation is much faster; instance creation and named attribute access are a bit f

[issue30859] Can't install Python for Windows 3.6.1 on multiple profiles

2017-07-05 Thread Joe Jacobs
Joe Jacobs added the comment: So, with the feedback, i uninstalled everything and tried installing again, but in the %appdata% folder for both users and everything worked fine. Both users have python installed. If i run the web launcher, the first time it installs on the second account and

[issue30859] Can't install Python for Windows 3.6.1 on multiple profiles

2017-07-05 Thread Joe Jacobs
Joe Jacobs added the comment: I checked, there are no install logs. Just realized, maybe an issue with the web installer rather than the base installer. python-3.6.1-amd64-webinstall Had downloaded this file and tried to run it. I wouldn't even get to the screen that asks you to

[issue30859] Can't install Python for Windows 3.6.1 on multiple profiles

2017-07-05 Thread Joe Jacobs
New submission from Joe Jacobs: Windows 7 Ultimate: SP1. Fully pathed as of July 4th, 2017 My Windows 7 install has multiple user accounts. The main install account is set up with "Administrator" privledges. Some point in the past I had installed Python 3.6.1 on the main account.

[issue13349] Non-informative error message in index() and remove() functions

2017-05-27 Thread Joe Jevnik
Changes by Joe Jevnik : -- pull_requests: +1919 ___ Python tracker <http://bugs.python.org/issue13349> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue30477] tuple.index error message improvement

2017-05-25 Thread Joe Jevnik
Joe Jevnik added the comment: As a more meta question: I have noticed that many error messages in the stdlib use PyErr_SetString, or choose to use the type name instead of the repr of the object. Is there a reason for this? I normally try to fill the error message with as much context as

[issue30477] tuple.index error message improvement

2017-05-25 Thread Joe Jevnik
Joe Jevnik added the comment: I agree, "%R in tuple" is enough information for me. This would also remove the need to manually repr the object. -- type: enhancement -> ___ Python tracker <http://bugs.pytho

[issue30477] tuple.index error message improvement

2017-05-25 Thread Joe Jevnik
New submission from Joe Jevnik: The old error of tuple.index(x): x not in tuple seemed very confusing to me. It was also harder to quickly understand what the program was doing wrong. This saves people a second pass through the program under the debugger because they can just see what the

[issue30180] PyArg_ParseTupleAndKeywords supports required keyword only arguments

2017-04-26 Thread Joe Jevnik
New submission from Joe Jevnik: I opened a pr to remove a line in the docs about $ needing to follow | in PyArg_ParseTupleAndKeywords. In practice, you can just use a $ to create required keyword arguments which intuitively makes sense. I was told this should raise a SystemError; however, you

[issue30180] PyArg_ParseTupleAndKeywords supports required keyword only arguments

2017-04-26 Thread Joe Jevnik
Changes by Joe Jevnik : -- assignee: -> docs@python components: +Documentation nosy: +docs@python ___ Python tracker <http://bugs.python.org/issue30180> ___ _

[issue29468] zipfile should catch ValueError as well as OSError to detect bad seek calls

2017-02-06 Thread Joe Jevnik
New submission from Joe Jevnik: In zipfile.py only OSError is checked to see if seek fails: ``` def _EndRecData64(fpin, offset, endrec): """ Read the ZIP64 end-of-archive records and use that to update endrec """ try: fpin.seek(offse

[issue29096] Signal Handlers reliably cause UnboundLocalErrors

2016-12-28 Thread Joe Jevnik
Joe Jevnik added the comment: The issue appears to be in ceval.c:unicode_concatenate (or the py2 equivalent) The code sets the local variable on the lhs to NULL before doing a potentially inplace append to the string. This means that if a signal is raised during the concat, we never hit the

[issue28254] Add C API for gc.enable, gc.disable, and gc.isenabled

2016-09-27 Thread Joe Jevnik
Joe Jevnik added the comment: I definitely could have used PyImport_Import and PyObject_Call to accomplish this task; however, when I looked at at the implementation of these functions it seemed like a lot of overhead and book keeping just to set a boolean. Since I was already in a C

[issue28254] Add C API for gc.enable, gc.disable, and gc.isenabled

2016-09-22 Thread Joe Jevnik
New submission from Joe Jevnik: I was writing an extension module that was working with weakrefs and wanted to ensure that the GC would not run for a block of code. I noticed that gc.enable/gc.disable are not exposed to C and the state of the gc is in a static variable so it cannot be mutated

[issue27241] Catch exceptions raised in pstats add (repl)

2016-08-02 Thread Joe Jevnik
Joe Jevnik added the comment: bump -- ___ Python tracker <http://bugs.python.org/issue27241> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue27467] distutils.config API different between <=3.5.1 and 3.5.2

2016-07-07 Thread Joe
New submission from Joe: In Python 3k releases leading up to 3.5.2, distutils.config imported "ConfigParser", but now imports "RawConfigParser" in the latest release. The documentation indicates "RawConfigParser" is considered legacy and "ConfigParser&quo

[issue27359] OrderedDict pseudo-literals (WIP)

2016-06-20 Thread Joe Jevnik
Joe Jevnik added the comment: Sorry about undoing the settings, that was unintentional. The UI around these settings is sort of unintuitive. I was waiting to propose this on python-dev until I got it working, but I wanted to push up what I did have as a work in progress so I could try to get

[issue27359] OrderedDict pseudo-literals (WIP)

2016-06-20 Thread Joe Jevnik
Joe Jevnik added the comment: Assuming all dicts are ordered, there will be no need for this (as long as the parser preserves order) so I am okay with dropping this. One of the main use cases I have for wanting nice literals is when I am working on tests. I find it very nice to be able to

[issue27359] OrderedDict pseudo-literals (WIP)

2016-06-20 Thread Joe Jevnik
New submission from Joe Jevnik: This proposes the following syntax for creating OrderedDict literals: OrderedDict[k1: v1, k2: v2, ...] This is implemented by putting a metaclass on OrderedDict which has a getitem that turns the slices into a list of tuples (after validation). The idea is

[issue27241] Catch exceptions raised in pstats add (repl)

2016-06-06 Thread Joe Jevnik
New submission from Joe Jevnik: I was trying to add a file and accidently mistyped the name which crashed the repl session. I think it would be nicer to report the failure but bring me back to the prompt so I can decide what I would like to do. This patch catches any IOErrors that can be

[issue27039] bytearray.remove cannot remove bytes with value greater than 127

2016-05-16 Thread Joe Jevnik
Joe Jevnik added the comment: This seems to just be a bug in the implementation of remove. I have a patch to fix this and a test case. -- keywords: +patch nosy: +ll Added file: http://bugs.python.org/file42875/bytearray-remove.patch

[issue25770] expose name, args, and kwargs from methodcaller

2016-05-04 Thread Joe Jevnik
Joe Jevnik added the comment: people have had some time to think about this, whats should the name be so we can merge this? -- ___ Python tracker <http://bugs.python.org/issue25

[issue26802] Avoid copy in call_function_var when no extra stack args are passed

2016-04-19 Thread Joe Jevnik
Joe Jevnik added the comment: CALL_FUNCTION doesn't use extended arg; I don't see what we get by adding some opcode to convert the sequence into a tuple. We also don't want to box up the stack arguments into a tuple because when we do a CALL_FUNCTION to a python function w

[issue26802] Avoid copy in call_function_var when no extra stack args are passed

2016-04-19 Thread Joe Jevnik
Joe Jevnik added the comment: > It's possible that stararg is already an empty tuple. Is it worth to use it? PyTuple_New(0) should return the unit tuple in most cases: #if PyTuple_MAXSAVESIZE > 0 if (size == 0 && free_list[0]) { op = free_list[0]; Py

[issue26802] Avoid copy in call_function_var when no extra stack args are passed

2016-04-19 Thread Joe Jevnik
Changes by Joe Jevnik : -- type: -> performance ___ Python tracker <http://bugs.python.org/issue26802> ___ ___ Python-bugs-list mailing list Unsubscrib

[issue26802] Avoid copy in call_function_var when no extra stack args are passed

2016-04-19 Thread Joe Jevnik
Joe Jevnik added the comment: in _PyEval_EvalCodeWithName we will have already pulled the underlying array out of the tuple subclass and the then we will reconstruct the vararg value in the locals from this array. This means that with this patch we can get: >>> class C(tup

[issue23926] skipitem() in getargs.c still supports 'w' and 'w#', and shouldn't

2016-04-18 Thread Joe Jevnik
Joe Jevnik added the comment: bump -- ___ Python tracker <http://bugs.python.org/issue23926> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue26802] Avoid copy in call_function_var when no extra stack args are passed

2016-04-18 Thread Joe Jevnik
New submission from Joe Jevnik: When star unpacking positions into a function we can avoid a copy iff there are no extra arguments coming from the stack. Syntactically this looks like: `f(*args)` This reduces the overhead of the call by about 25% (~30ns on my machine) based on some light

[issue26448] dis.findlabels ignores EXTENDED_ARG

2016-02-26 Thread Joe Jevnik
Changes by Joe Jevnik : -- nosy: +ll ___ Python tracker <http://bugs.python.org/issue26448> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue26379] zlib decompress as_bytearray flag

2016-02-18 Thread Joe Jevnik
Joe Jevnik added the comment: The recipe you showed looks like it is very complicated for the expected use case of decompressing all of the data into a mutable buffer. One difference I see with this and struct.pack_into or socket.recv_into is that in both of those cases we know how large our

[issue25770] expose name, args, and kwargs from methodcaller

2016-02-17 Thread Joe Jevnik
Joe Jevnik added the comment: ping: any decision on this? -- ___ Python tracker <http://bugs.python.org/issue25770> ___ ___ Python-bugs-list mailing list Unsub

[issue26379] zlib decompress as_bytearray flag

2016-02-17 Thread Joe Jevnik
New submission from Joe Jevnik: Adds a keyword only flag to zlib.decompress and zlib.Decompress.decompress which marks that the return type should be a bytearray instead of bytes. The use case for this is reading compressed data into a mutable array to do further operations without requiring

[issue24379] operator.subscript

2015-12-22 Thread Joe Jevnik
Joe Jevnik added the comment: > and the latter might give the impression it was some sort of special > method/attribute when it was not. Wouldn't adding this be special because it is specifically reserved by CPython as an implementation detail? Also, would adding this name

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-22 Thread Joe Jevnik
Joe Jevnik added the comment: Is there a decision on the name? I can update the patch if needed. -- ___ Python tracker <http://bugs.python.org/issue25770> ___ ___

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-02 Thread Joe Jevnik
Joe Jevnik added the comment: partial's unique usage is why I feel like it would be so jarring for them do be named differently. I would be happiest having this feature at all so I will change the name to 'kwargs' if you would like. I just want to be sure that my reasons fo

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-01 Thread Joe Jevnik
Joe Jevnik added the comment: I agree that it is a strange name and I also think that it could be immutable or a copy of the internal dict; however, I think that consistency with existing APIs in the standard library is more important. 'keywords' is still very clear in context and

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-01 Thread Joe Jevnik
Joe Jevnik added the comment: I only want this feature to keep the usage close to functools.partial. I was actually sort of surprised to see that mutation of the dict held in partial, but I would rather be consistent. -- ___ Python tracker <h

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-01 Thread Joe Jevnik
Joe Jevnik added the comment: Added a test case for the mutation of keywords. -- Added file: http://bugs.python.org/file41204/methodcaller-attrs-2.patch ___ Python tracker <http://bugs.python.org/issue25

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-01 Thread Joe Jevnik
Joe Jevnik added the comment: Thanks for pointing me at the refleak, uploading an update -- Added file: http://bugs.python.org/file41203/methodcaller-attrs-1.patch ___ Python tracker <http://bugs.python.org/issue25

[issue25770] expose name, args, and kwargs from methodcaller

2015-12-01 Thread Joe Jevnik
Joe Jevnik added the comment: Thanks for review, looking into the reference count issue. -- ___ Python tracker <http://bugs.python.org/issue25770> ___ ___ Pytho

[issue25770] expose name, args, and kwargs from methodcaller

2015-11-30 Thread Joe Jevnik
New submission from Joe Jevnik: This patch adds 3 properties to methodcaller objects for inspecting the object at runtime: 1. 'name': the name of the method to call 2. 'args': the position arguments to pass to the method 3. 'keywords': the keyword arguments t

[issue23926] skipitem() in getargs.c still supports 'w' and 'w#', and shouldn't

2015-10-20 Thread Joe Jevnik
Joe Jevnik added the comment: bumping this issue -- ___ Python tracker <http://bugs.python.org/issue23926> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue24379] operator.subscript

2015-08-16 Thread Joe Jevnik
Joe Jevnik added the comment: Sorry about the ideas thread. Thank you for merging this! -- ___ Python tracker <http://bugs.python.org/issue24379> ___ ___ Pytho

[issue24379] operator.subscript

2015-07-28 Thread Joe Jevnik
Joe Jevnik added the comment: I posted this to python Ideas on June 10th of this year. That is where we decided to rename it from `slice.literal` to `operator.subscript`. -- ___ Python tracker <http://bugs.python.org/issue24

[issue23926] skipitem() in getargs.c still supports 'w' and 'w#', and shouldn't

2015-07-25 Thread Joe Jevnik
Joe Jevnik added the comment: bumping so that we don't forget about this. -- ___ Python tracker <http://bugs.python.org/issue23926> ___ ___ Python-bugs-list m

[issue24379] operator.subscript

2015-07-24 Thread Joe Jevnik
Joe Jevnik added the comment: Any more comments? -- ___ Python tracker <http://bugs.python.org/issue24379> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue23926] skipitem() in getargs.c still supports 'w' and 'w#', and shouldn't

2015-07-15 Thread Joe Jevnik
Joe Jevnik added the comment: Sorry it took so long to get back to this. I didn't realize this was still open. I have provided the update to the docs and moved it to the 3.6 section. I also made sure the patch still applies and the tests all pass. -- Added file: http://bugs.pytho

[issue24379] operator.subscript

2015-07-14 Thread Joe Jevnik
Joe Jevnik added the comment: I updated the docstring -- ___ Python tracker <http://bugs.python.org/issue24379> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue24379] operator.subscript

2015-07-12 Thread Joe Jevnik
Changes by Joe Jevnik : Added file: http://bugs.python.org/file39913/operator_subscript_pyonly.patch ___ Python tracker <http://bugs.python.org/issue24379> ___ ___ Pytho

[issue24379] operator.subscript

2015-07-12 Thread Joe Jevnik
Changes by Joe Jevnik : Added file: http://bugs.python.org/file39907/operator_subscript_pyonly.patch ___ Python tracker <http://bugs.python.org/issue24379> ___ ___ Pytho

[issue24379] operator.subscript

2015-07-12 Thread Joe Jevnik
Joe Jevnik added the comment: is it normal to get a lot of 500s when using the review system? -- Added file: http://bugs.python.org/file39906/operator_subscript_pyonly.patch ___ Python tracker <http://bugs.python.org/issue24

[issue24379] operator.subscript

2015-07-11 Thread Joe Jevnik
Joe Jevnik added the comment: updating to address the docs and order of assertions -- Added file: http://bugs.python.org/file39904/operator_subscript_pyonly.patch ___ Python tracker <http://bugs.python.org/issue24

[issue24379] operator.subscript

2015-07-11 Thread Joe Jevnik
Joe Jevnik added the comment: updating with the slots change This also adds a ton of test cases -- Added file: http://bugs.python.org/file39902/operator_subscript_pyonly.patch ___ Python tracker <http://bugs.python.org/issue24

[issue24379] operator.subscript

2015-07-11 Thread Joe Jevnik
Joe Jevnik added the comment: Ah, I hadn't seen that, I will address those now, sorry about that. -- ___ Python tracker <http://bugs.python.org/is

[issue24379] operator.subscript

2015-07-11 Thread Joe Jevnik
Joe Jevnik added the comment: ping: is there anything blocking this? -- ___ Python tracker <http://bugs.python.org/issue24379> ___ ___ Python-bugs-list mailin

[issue24379] operator.subscript

2015-06-23 Thread Joe Jevnik
Joe Jevnik added the comment: I removed the C implementation. -- Added file: http://bugs.python.org/file39792/operator_subscript_pyonly.patch ___ Python tracker <http://bugs.python.org/issue24

[issue24379] operator.subscript

2015-06-22 Thread Joe Jevnik
Joe Jevnik added the comment: I just moved it over since I implemented it for slice originally, I can drop the C implementation. -- ___ Python tracker <http://bugs.python.org/issue24

[issue24379] operator.subscript

2015-06-22 Thread Joe Jevnik
Joe Jevnik added the comment: Based on some discussion on python-ideas, this is being renamed to operator.subscript. Here is the patch that makes the correct the changes to move this object into the operator module. -- components: +Extension Modules -Interpreter Core title

[issue24379] slice.literal notation

2015-06-10 Thread Joe Jevnik
Joe Jevnik added the comment: It is a singleton, does not accept the `maketuple` flag, and is written in C. I did not know about the s_ attribute of numpy before writing this; however, I still think that moving this object to slice improves code clarity (s_ is not a super clear name). I also

[issue24379] slice.literal notation

2015-06-10 Thread Joe Jevnik
Joe Jevnik added the comment: >>> slice.literal[0] 0 >>> y = slice.literal[1:2] slice(1, 2, None) >>> slice.literal[0:1, ..., 3] (slice(0, 1, None), Ellipsis, 3) The way this object works right now does not create instances of some inner class of slice, instead,

[issue24379] slice.literal notation

2015-06-09 Thread Joe Jevnik
Joe Jevnik added the comment: > What I'm missing is a way to use such an object to actually index/slice > something Sorry, I am not sure I understand what you mean by this? You can pass a slice object, or a tuple of slices in subscript notation. >>> [1, 2, 3, 4][slice(2)]

  1   2   3   >