[issue29427] Option to skip padding for base64 urlsafe encoding/decoding

2017-08-06 Thread Brian Thorne
Brian Thorne added the comment: Hi Robert, It would be at least a week or two before I could take another look at this so please feel free to work on it. Not sure why I didn't write a patch at the time! -- ___ Python tracker

[issue29427] Option to skip padding for base64 urlsafe encoding/decoding

2017-02-02 Thread Brian Thorne
New submission from Brian Thorne: Suggest changing base64 module to better handle encoding schemes that don't use padding. Because RFC4648 [1] allows other RFCs that implement RFC4648-compliant base64url encoding to explicitly stipulate that there is no padding. Dropping the paddi

[issue16192] ctypes - documentation example

2015-01-08 Thread Brian Thorne
Changes by Brian Thorne : -- nosy: -Thorney, chris.jerdonek, eric.araujo, ezio.melotti, georg.brandl status: open -> closed ___ Python tracker <http://bugs.python.org/issu

[issue18511] random.shuffle could be faster

2013-07-19 Thread Brian Thorne
Brian Thorne added the comment: Just did some testing on 2.7 and 3.3 on Windows and Ubuntu, the speedup is just noticeable - but much less so as the list grows. -- nosy: +Thorney ___ Python tracker <http://bugs.python.org/issue18

[issue15359] Sockets support for CAN_BCM

2013-01-22 Thread Brian Thorne
Brian Thorne added the comment: I've added (some) docs and added checking of the BCM constants to the test_socket module. I would guess that checking each broadcast manager function provided by the kernel isn't required? -- Added file: http://bugs.python.org/file28805/

[issue15359] Sockets support for CAN_BCM

2013-01-16 Thread Brian Thorne
Brian Thorne added the comment: Thanks for the review Charles-François. I can't make your suggested bcm_msg_fmt work - it was deadlocking on my machine! -- Added file: http://bugs.python.org/file28756/bcm3.patch ___ Python tracker

[issue15359] Sockets support for CAN_BCM

2013-01-15 Thread Brian Thorne
Brian Thorne added the comment: I've added a single BCM test. Any feedback on it would be good and then I'll add more complex ones. Should the documentation be updated in this patch as well? -- Added file: http://bugs.python.org/file28747/

[issue12428] functools test coverage

2012-11-13 Thread Brian Thorne
Brian Thorne added the comment: > * Why the try block when there wasn't one before? > * Should reduce be added to __all__ only conditionally? My mistake, the try block should have just covered the import of partial - that is after all the exceptional circumstance we can deal with b

[issue16192] ctypes - documentation example

2012-10-11 Thread Brian Thorne
Brian Thorne added the comment: > The very beginning of the ctypes documentation has documentation to that > effect >Does that address your suggestion? Thanks Chris, that explains why the difference is present on non 32 bit platforms. Would using c_long for the examples produce the

[issue16192] ctypes - documentation example

2012-10-10 Thread Brian Thorne
New submission from Brian Thorne: The Python docs for ctypes have embedded examples including: >>> c_int() c_long(0) But on my machine I get the (more expected?): >>> c_int() c_int(0) Perhaps if some machines expect otherwise that should be documented, otherwise mi

[issue4508] distutils compiler not handling spaces in path to output/src files

2012-08-05 Thread Brian Thorne
Brian Thorne added the comment: I've put together a patch adding the test requested. There is no problem on my Ubuntu machine with python 3.3. There is a comment in the file saying Don't load the xx module more than once, I am unsure whether my patch (using a renamed c file) vio

[issue12428] functools test coverage

2012-08-05 Thread Brian Thorne
Brian Thorne added the comment: Back to a simpler closure implementation of partial and skip the repr test for python implementation. -- Added file: http://bugs.python.org/file26697/functools.patch ___ Python tracker <http://bugs.python.

[issue12428] functools test coverage

2012-07-23 Thread Brian Thorne
Brian Thorne added the comment: Thanks Antoine, the __class__ attribute wasn't useful, I've removed that. Overriding the __setattr__ and __delattr__ gives consistent behaviour with the both versions - allowing the unittest reuse. Also I've changed thread back to _thre

[issue12428] functools test coverage

2012-07-16 Thread Brian Thorne
Changes by Brian Thorne : -- nosy: +jackdied ___ Python tracker <http://bugs.python.org/issue12428> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue15359] Sockets support for CAN_BCM

2012-07-15 Thread Brian Thorne
Brian Thorne added the comment: Once I've got more complete examples, I can update the patch to include a testcase. -- ___ Python tracker <http://bugs.python.org/is

[issue15359] Sockets support for CAN_BCM

2012-07-15 Thread Brian Thorne
New submission from Brian Thorne : In addition to CAN_RAW introduced in Python 3.3, it would be really useful to expose the CAN_BCM protocol. Effectively it hands off as much to the kernel as possible which gives Python programs the ability to send and receive many periodic messages with

[issue12428] functools test coverage

2012-04-11 Thread Brian Thorne
Brian Thorne added the comment: I've updated the patch to address the comments here and in the code review. I added more cross testing of the pure Python implementation of partial - as you pointed out inheritance wasn't supported so I changed from the simple closure to a class impl

[issue12428] functools test coverage

2011-07-29 Thread Brian Thorne
Brian Thorne added the comment: Cheers for the comments Eric. I've modified the patch accordingly. -- Added file: http://bugs.python.org/file22801/functools2.diff ___ Python tracker <http://bugs.python.org/is

[issue12428] functools test coverage

2011-06-28 Thread Brian Thorne
New submission from Brian Thorne : The test coverage for functools was down around ~60%, this is a patch to bring that up to ~98%. Made two changes to the Lib/functools.py file itself: 1) Moved the Python implementation of partial into Lib/functools.py from Lib/test/test_functools.py which

[issue4106] multiprocessing occasionally spits out exception during shutdown

2011-01-18 Thread Brian Thorne
Brian Thorne added the comment: With the example script attached I see the exception every time. On Ubuntu 10.10 with Python 2.6 Since the offending line in multiprocesing/queues.py (233) is a debug statement, just commenting it out seems to stop this exception. Looking at the util file

[issue4508] distutils compiler not handling spaces in path to output/src files

2010-08-24 Thread Brian Thorne
Brian Thorne added the comment: Hopefully this works! Otherwise would a tar.gz or a zip archive be ok? -- Added file: http://bugs.python.org/file18640/4508example.diff ___ Python tracker <http://bugs.python.org/issue4

[issue4508] distutils compiler not handling spaces in path to output/src files

2010-08-24 Thread Brian Thorne
Changes by Brian Thorne : Removed file: http://bugs.python.org/file18638/example.diff ___ Python tracker <http://bugs.python.org/issue4508> ___ ___ Python-bugs-list mailin

[issue4508] distutils compiler not handling spaces in path to output/src files

2010-08-24 Thread Brian Thorne
Brian Thorne added the comment: Sorry, here is the example as a diff against an empty directory. (I'm 100% sure the Ubuntu archiving program that I just used is free software though...) -- Added file: http://bugs.python.org/file18638/example

[issue4508] distutils compiler not handling spaces in path to output/src files

2010-08-24 Thread Brian Thorne
Brian Thorne added the comment: Comments on the SciPy-Dev mailing list: >>> While testing Scipy's weave on several different Windows installations, I came across some problems with spaces in paths that often prevent weave from working. I can see a change that could probably get

[issue4508] distutils compiler not handling spaces in path to output/src files

2009-01-11 Thread Brian Thorne
Brian Thorne added the comment: Ok - this bug has nothing to do with weave itself. I think any extension written in C or C++ that has a space in the filename will raise the error in both windows and gnu/linux. Here is a simple c extension with and without spaces. The one with spaces in the

[issue4508] distutils compiler not handling spaces in path to output/src files

2008-12-03 Thread Brian Thorne
New submission from Brian Thorne <[EMAIL PROTECTED]>: I found this bug when using scipy's weave and having some inline C/C++ code. Weave uses distutils to compile the source in its own directory, python was installed under "program files". Unfortunately it doesn't se