[issue4476] compileall fails if current dir has a "types" package

2021-07-31 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- pull_requests: -26023 ___ Python tracker <https://bugs.python.org/issue4476> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue4476] compileall fails if current dir has a "types" package

2021-07-31 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: sorry about the noise, everyone. Missed an '6' in the end of the issue name -- ___ Python tracker <https://bugs.python.org/issue4

[issue44766] [easy doc] Remove redundant info in README.valgrind

2021-07-31 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- keywords: +patch nosy: +fbidu nosy_count: 2.0 -> 3.0 pull_requests: +26026 stage: -> patch review pull_request: https://github.com/python/cpython/pull/27509 ___ Python tracker <https://bugs.python.org/i

[issue4476] compileall fails if current dir has a "types" package

2021-07-31 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- nosy: +fbidu nosy_count: 5.0 -> 6.0 pull_requests: +26023 pull_request: https://github.com/python/cpython/pull/27509 ___ Python tracker <https://bugs.python.org/iss

[issue44539] Imghdr JPG Quantized

2021-07-03 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- nosy: +fbidu ___ Python tracker <https://bugs.python.org/issue44539> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue37195] test_utime fails on MacOS Mojave (Kernel Version 18.6.0:)

2021-07-03 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: Hi all! I'm seeing the same error that @pablogsal saw here, but I'm on Linux Mint 20.1 x86_64 (Kernel 5.4.0-77): ./python -m test test_os -R : -v == CPython 3.11.0a0 (heads/pr_26964:d375c08c75, Jul 3 2021, 07:47:01) [GCC 9.3.0] == Linux-5.4.0-77-generic

[issue42914] pprint numbers with underscore

2021-03-09 Thread Felipe
Felipe added the comment: All yours! I'm tied up so won't be able to submit the PR On Thu, 25 Feb 2021 at 10:12, Stéphane Blondon wrote: > > Stéphane Blondon added the comment: > > I add the same idea but later than you, so I'm interested by such feature. > > Felipe:

[issue43019] wait_for(to_thread)) does not work as expected. Extra documentation or fix needed.

2021-01-24 Thread Felipe Faria
New submission from Felipe Faria : Consider the following: ``` import asyncio import time async def async_test(): while True: await asyncio.sleep(1) print("in here - async") def sync_test(): while True: time.sleep(1) print("in here - sy

[issue42914] pprint numbers with underscore

2021-01-12 Thread Felipe
Felipe added the comment: Here is an implementation of the safe repr for numbers if helpful: ``` def safe_repr_int(object): sign = '' if object < 0: sign = '-' object = -object r = repr(object) if len(r) <= 4: return sign + r parts =

[issue42914] pprint numbers with underscore

2021-01-12 Thread Felipe
New submission from Felipe : It would be nice if pprint learned to insert underscores in long numbers Current behavior: >>> pprint.pprint(int(1e9)) 10 Desired behavior >>> pprint.pprint(int(1e9)) 1_000_000_000 Wikipedia tells me that "groups of 3" i

[issue21032] Socket leak if HTTPConnection.getresponse() fails

2020-10-23 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- nosy: +fbidu nosy_count: 3.0 -> 4.0 pull_requests: +21848 pull_request: https://github.com/python/cpython/pull/22737 ___ Python tracker <https://bugs.python.org/issu

[issue13464] HTTPResponse is missing an implementation of readinto

2020-10-23 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- nosy: +fbidu nosy_count: 5.0 -> 6.0 pull_requests: +21847 pull_request: https://github.com/python/cpython/pull/22737 ___ Python tracker <https://bugs.python.org/issu

[issue42062] Usage of HTTPResponse.url

2020-10-17 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- keywords: +patch pull_requests: +21701 stage: -> patch review pull_request: https://github.com/python/cpython/pull/22738 ___ Python tracker <https://bugs.python.org/issu

[issue42062] Usage of HTTPResponse.url

2020-10-17 Thread Felipe Rodrigues
New submission from Felipe Rodrigues : Hello all, While testing some static analysis tools on HTTP/client.py, Pylint pointed me to HTTPResponse.geturl() method with a "no-member" error for the `url` attribute. I tried invoking the `geturl` method and reading the `HTTPResponse.url`

[issue42060] Usage of assert in http/client.py

2020-10-17 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- keywords: +patch pull_requests: +21700 stage: -> patch review pull_request: https://github.com/python/cpython/pull/22737 ___ Python tracker <https://bugs.python.org/issu

[issue42060] Usage of assert in http/client.py

2020-10-17 Thread Felipe Rodrigues
New submission from Felipe Rodrigues : Hi all! I was testing some static analysis tool and decided to use the HTTP module as testing ground. While running `bandit` at the client module, it detected 3 instances of using `assert` inside the code. Twice in the HTTPResponse class and once

[issue37943] mimetypes.guess_extension() doesn’t get JPG right

2020-08-22 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: @_savage, on the commit @xtreak referred, there's a note that "image/jpg" and some other non-standard mimetypes are only supported if `strict=False`[1] So, this: >>> mimetypes.guess_extension("image/jpg") Gi

[issue41220] add optional make_key argument to lru_cache

2020-07-10 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: Correction: (...) on the _caller_ to solve the hashing* issue (...) -- ___ Python tracker <https://bugs.python.org/issue41

[issue41220] add optional make_key argument to lru_cache

2020-07-10 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: Hello all, I love discussions about caching! While I do see the point of Itayazolay's proposal, I think that it should be on the _caller_ to solve the caching issue, and not on the _callee_ to provide a way to make it happen. That is, I think that if one

[issue35919] multiprocessing: shared manager Pool fails with AttributeError

2020-07-09 Thread Felipe A. Hernandez
Change by Felipe A. Hernandez : -- components: +Library (Lib) type: -> behavior versions: +Python 3.5, Python 3.6, Python 3.7, Python 3.8, Python 3.9 ___ Python tracker <https://bugs.python.org/issu

[issue35919] multiprocessing: shared manager Pool fails with AttributeError

2020-07-09 Thread Felipe A. Hernandez
Felipe A. Hernandez added the comment: import traceback import multiprocessing.managers class MyManager(multiprocessing.managers.SyncManager): pass class DictList(multiprocessing.managers.BaseProxy): _method_to_typeid_ = {'__getitem__': 'dict'} def __getitem__(self, key

[issue40042] Enum Flag: psuedo-members have None for name attribute

2020-05-18 Thread Felipe Rodrigues
New submission from Felipe Rodrigues : Hi, Can you elaborate on this? Thanks! -- nosy: +fbidu ___ Python tracker <https://bugs.python.org/issue40042> ___ ___

[issue39246] shutil.rmtree is inefficient because of using os.scandir instead of os.walk

2020-01-08 Thread Felipe A. Hernandez
Felipe A. Hernandez added the comment: After some tests, due the accumulating nature of fwalk, I've just realised it's not very safe for big directories, so I'll be closing this issue. Alternatively, using py37+ fd based scandir, and dir_fd unlink and rmdir calls would reduce complexity

[issue39246] shutil.rmtree is inefficient because of using os.scandir instead of os.walk

2020-01-07 Thread Felipe A. Hernandez
New submission from Felipe A. Hernandez : os.rmtree has fd-based symlink replacement protection when iterating with scandir (after bpo-28564). This logic could be greatly simplified simply by os.fwalk in supported platforms, which already implements a similar (maybe safer) protection

[issue38619] [Doc] UUID.hex is lowercase

2019-10-28 Thread Felipe
New submission from Felipe : The hex property of `UUID` is implemented as `'%032x' % self.int` Is it specified that this will always be lowercase? If so, can we add a note to the documentation indicating so? -- assignee: docs@python components: Documentation, Library (Lib) messages

[issue19953] __iadd__() doc not strictly correct

2019-02-06 Thread Felipe Manzano
Change by Felipe Manzano : -- pull_requests: +11750, 11751 ___ Python tracker <https://bugs.python.org/issue19953> ___ ___ Python-bugs-list mailing list Unsub

[issue19953] __iadd__() doc not strictly correct

2019-02-06 Thread Felipe Manzano
Change by Felipe Manzano : -- pull_requests: +11750 ___ Python tracker <https://bugs.python.org/issue19953> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue19953] __iadd__() doc not strictly correct

2019-02-06 Thread Felipe Manzano
Change by Felipe Manzano : -- pull_requests: +11750, 11751, 11752 ___ Python tracker <https://bugs.python.org/issue19953> ___ ___ Python-bugs-list mailin

[issue23078] unittest.mock patch autospec doesn't work on staticmethods

2019-01-17 Thread Felipe
Felipe added the comment: Please go ahead with the PR. I can't push this one through, but would be great to have this finally land! On Thu, 17 Jan 2019 at 03:42, Karthikeyan Singaravelan < rep...@bugs.python.org> wrote: > > Karthikeyan Singaravelan added the comment: > >

[issue25416] Add encoding aliases from the (HTML5) Encoding Standard

2018-11-01 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: Ezio, I have issued a simple PR that adds just the two aliases cited in the issue's initial message. I would like to implement tests but as I wrote in the PR's message, I'm not really sure how to proceed with that. bpo-18624 is really related

[issue19376] document that strptime() does not support the Feb 29 if the format does not contain the year

2018-10-30 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: What about adding some more information in the exception message in addition to the docs? I mean, if we're raising an issue because someone inserted Feb 29, we add a note about leap year in the exception message -- nosy: +fbidu

[issue18624] Add alias for iso-8859-8-i which is the same as iso-8859-8

2018-10-30 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- pull_requests: +9550 ___ Python tracker <https://bugs.python.org/issue18624> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue25416] Add encoding aliases from the (HTML5) Encoding Standard

2018-10-30 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- keywords: +patch pull_requests: +9549 stage: needs patch -> patch review ___ Python tracker <https://bugs.python.org/issu

[issue26005] Denial of Service in SimpleHTTPServer and BaseHTTPServer

2018-10-05 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- pull_requests: +9104 ___ Python tracker <https://bugs.python.org/issue26005> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34576] [EASY doc] http.server, SimpleHTTPServer: warn users on security

2018-10-05 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- keywords: +patch pull_requests: +9103 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue34576> ___ _

[issue34832] "Short circuiting" in base64's b64decode, decode, decodebytes

2018-09-29 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: For reference in future discussions, Python's base64 module implements RFC 3548 (https://tools.ietf.org/html/rfc3548) whose section 2.3 (https://tools.ietf.org/html/rfc3548#section-2.3) discusses about "Interpretation of non-alphabet characters in en

[issue34832] "Short circuiting" in base64's b64decode, decode, decodebytes

2018-09-28 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: Actually, I'm not even sure if it makes sense to decode the 'first valid substring'... IMHO, we should warn the user -- ___ Python tracker <https://bugs.python.org/issue34

[issue34832] "Short circuiting" in base64's b64decode, decode, decodebytes

2018-09-28 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: I am not sure if simply ignoring the non-valid character is the best way to go. Feels like silencing errors. b64decode does accept the 'validate' flag - defaulted to False - that will halt the execution and throw an error. What might be a good idea

[issue34576] [EASY doc] http.server, SimpleHTTPServer: warn users on security

2018-09-27 Thread Felipe Rodrigues
Felipe Rodrigues added the comment: Well, even if we do fix some security issues in SimpleHTTPServer, it doesn't change the fact that it shouldn't really be used for sensitive applications. I like how Django docs handles a similar issue regarding their development server (https

[issue34728] deprecate *loop* argument for asyncio.sleep

2018-09-19 Thread Felipe Rodrigues
Change by Felipe Rodrigues : -- nosy: +fbidu ___ Python tracker <https://bugs.python.org/issue34728> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue32432] [BUG] Python vs Macbook High Sierra 10.13.2

2017-12-28 Thread Felipe Filgueira Barral
Felipe Filgueira Barral <felipef...@gmail.com> added the comment: Tks Ronald, solve the problem! =) -- ___ Python tracker <rep...@bugs.python.org> <https://bugs.python

[issue32432] [BUG] Python vs Macbook High Sierra 10.13.2

2017-12-28 Thread Felipe Filgueira Barral
Change by Felipe Filgueira Barral <felipef...@gmail.com>: -- stage: -> resolved status: open -> closed ___ Python tracker <rep...@bugs.python.org> <https://bugs.

[issue32432] [BUG] Python vs Macbook High Sierra 10.13.2

2017-12-27 Thread Felipe Filgueira Barral
New submission from Felipe Filgueira Barral <felipef...@gmail.com>: When i open the IDLE and try include ' or " in any script, the python closes alone! Does anyone know what might be happening? I use mac, version high sierra... Process: Python [1048] Path: /Applications/Python 3

[issue25144] 3.5 Win install fails with "TARGETDIR"

2017-11-20 Thread Felipe
Change by Felipe <felipe.nospam.oc...@gmail.com>: -- nosy: -fov ___ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue25144> ___ __

[issue31533] Dead link in SSLContext.set_ciphers documentation

2017-09-20 Thread Felipe
Felipe added the comment: Sure, will do! -- ___ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue31533> ___ ___ Python-bugs-list

[issue31533] Dead link in SSLContext.set_ciphers documentation

2017-09-20 Thread Felipe
Felipe added the comment: Woops. Apologies -- I somehow deleted that part from the report. Thanks Antoine for jumping in. If you are using the wiki url, make sure not to include the question mark at the end :-) -- ___ Python tracker <

[issue31533] Dead link in SSLContext.set_ciphers documentation

2017-09-20 Thread Felipe
New submission from Felipe: The link to the `OpenSSL cipher list format` 404s. I don't know if it's where the original meant to point, but the same information is available at https://wiki.openssl.org/index.php/Manual:Ciphers(1)#CIPHER_LIST_FORMAT -- assignee: docs@python components

[issue25785] TimedRotatingFileHandler missing rotations

2015-12-02 Thread Felipe Cruz
New submission from Felipe Cruz: I'm using TimedRotatingFileHandler to rotate a log file *every* minute. If I stop my program, in the middle of a minute, and start again, the next rotation will happen at (currentTime + 60). The result of this behavior is that I'll end up with files "l

[issue25432] isinstance documentation: explain behavior when type is a tuple

2015-10-26 Thread Felipe Volpone
Felipe Volpone added the comment: r.david.murray: I liked your way. -- ___ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue25432> ___ ___

[issue25144] 3.5 Win install fails with "TARGETDIR"

2015-09-17 Thread Felipe
Changes by Felipe <felipe.nospam.oc...@gmail.com>: Added file: http://bugs.python.org/file40492/Python 3.5.0 (32-bit)_20150916132422.log ___ Python tracker <rep...@bugs.python.org> <http://bugs.python

[issue25144] 3.5 Win install fails with "TARGETDIR"

2015-09-16 Thread Felipe
New submission from Felipe: The 3.5 Windows installer fails with "The TARGETDIR variable must be provided when invoking this installer" Here are the steps I followed: 1. Initial screen: - uncheck the add path and all users options - select "Customize installation"

[issue23078] unittest.mock patch autospec doesn't work on staticmethods

2015-09-14 Thread Felipe
Felipe added the comment: The attached patch implements these changes through _callable instead. This patch also ensures that the underlying object that staticmethod and classmethod wrap is a callable object as well. -- Added file: http://bugs.python.org/file40470/issue23078.patch

[issue24651] Mock.assert* API is in user namespace

2015-07-21 Thread Felipe
Felipe added the comment: Not sure it's my place to comment here, but here are my 2 cents: I think Robert's proposal to have module functions is the only way to have a user-friendly and robust API, and it solves more than just the assert typo problem. (And yes, it would require moving

[issue23078] unittest.mock patch autospec doesn't work on staticmethods

2015-06-02 Thread Felipe
Felipe added the comment: Regarding Claudiu's comment about `staticmethod(x)` or `classmethod(x)` not being callable, would it suffice to add a specific check of the form `(isinstance(x, (classmethod, staticmethod)) and _callable(x.__func__))`? Separately, would it be better to include

[issue22988] No error when yielding from `finally`

2014-12-03 Thread Felipe
New submission from Felipe: This bug report is the opposite of issue #14718. The interpreter did not raise an error when it encountered a `yield` expression inside the `finally` part of a `try/finally` statement. My system's info: Python 3.4.2 (v3.4.2:ab2c023a9432, Oct 6 2014, 22:15:05

[issue22988] No error when yielding from `finally`

2014-12-03 Thread Felipe
Felipe added the comment: Thanks for the clarification; sorry I misread issue #14718. I agree with Ethan's point. Though I would say Yield expressions are allowed anywhere in try ... except ... finally constructs. I'd also like to explicitly add a point about the exception-handling machinery

[issue20768] pyconfig.h #defines macros in global namespace

2014-02-25 Thread Felipe Sateler
New submission from Felipe Sateler: I reported the following in the debian bug tracker[1], and it was requested that I report it here. pyconfig.h has definitions like the following: #define HAVE_DIRENT_H 1 #define HAVE_DLFCN_H 1 These are the general form feature test macros take

[issue20768] pyconfig.h #defines macros in global namespace

2014-02-25 Thread Felipe Sateler
Felipe Sateler added the comment: I'm sorry but I definitely don't have time or knowledge about python to propose a patch (simply removing pyconfig.h clearly doesn't work). As to the question, please clarify. I have a python module, which includes Python.h, which includes pyconfig.h. I don't

[issue16105] Pass read only FD to signal.set_wakeup_fd

2013-08-16 Thread Felipe Cruz
Felipe Cruz added the comment: Looks good. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16105 ___ ___ Python-bugs-list mailing list

[issue16105] Pass read only FD to signal.set_wakeup_fd

2013-08-15 Thread Felipe Cruz
Felipe Cruz added the comment: Looks like PyErr_WriteUnraisable can be a better choice. Exceptions at random execution points looks a little bit dirty at least for this case. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org

[issue16853] add a Selector to the select module

2013-01-07 Thread Felipe Cruz
Felipe Cruz added the comment: Hi.. 2 comments related to Kqueue/OSX(16.8) 1 - In tulip/selectors.py L311 and L314 - is key.fd not fd 2 - In EventLoopTestsMixin::test_writer_callback if the writer socket isn't non-blocking, the test hangs forever.. 3 - Errors: ERROR: testCreateSslTransport

[issue16853] add a Selector to the select module

2013-01-03 Thread Felipe Cruz
Felipe Cruz added the comment: I think you have a point. Did you know about the tulip project? http://code.google.com/p/tulip/source/browse/tulip/unix_events.py#76 It has a PollsterBase class and a SelectPollster(PollsterBase) so the idea is to have a Poller(and you call poll()) but select

[issue16853] add a Selector to the select module

2013-01-03 Thread Felipe Cruz
Felipe Cruz added the comment: Hi Antonie, What you said also makes sense to me. There is one problem(?) that _map_events() is called for every event(for Poll and EPoll) and this can make things slower (I didn't tested it). Also, does it needs to be thread-safe

[issue16565] Increase Py_AddPendingCall array size

2012-11-29 Thread Felipe Cruz
Felipe Cruz added the comment: Running test_aio_read [New Thread 0x77ff7700 (LWP 20681)] [New Thread 0x761ff700 (LWP 20682)] Program received signal SIGSEGV, Segmentation fault. [Switching to Thread 0x761ff700 (LWP 20682)] sem_post () at ../nptl/sysdeps/unix/sysv/linux/x86_64

[issue16565] Increase Py_AddPendingCall array size

2012-11-29 Thread Felipe Cruz
Felipe Cruz added the comment: Yes! 2.7.3 build with pydebug. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16565 ___ ___ Python-bugs-list

[issue16565] Increase Py_AddPendingCall array size

2012-11-29 Thread Felipe Cruz
Felipe Cruz added the comment: Without debug backtrace #0 sem_post () at ../nptl/sysdeps/unix/sysv/linux/x86_64/sem_post.S:34 #1 0x004d456e in PyGILState_Release () #2 0x759b9cdc in notify_func_wrapper (arg=0x78c0) at ../sysdeps/pthread/aio_notify.c:45 #3

[issue16565] Increase Py_AddPendingCall array size

2012-11-28 Thread Felipe Cruz
Felipe Cruz added the comment: Just confirmed that signals is not a viable option. Is too slow, as Antonie already pointed. It's almost 5 times slower than with SIGEV_THREAD. The problem now is: If I use Py_AddPendingCall, the tests can hang sometimes. I can still follow Amaury suggestion

[issue16565] Increase Py_AddPendingCall array size

2012-11-27 Thread Felipe Cruz
New submission from Felipe Cruz: Current pending calls limit is too small and it can be easily reached in very intensive async file io applications. There is a little hack in pyaio[1] which sleeps if Py_AddPendingCall returns 0 but It's not totally clear to me why the size of pendind calls

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-16 Thread Felipe Cruz
Felipe Cruz added the comment: I've followed latest suggestions. Test and code updated. -- Added file: http://bugs.python.org/file27598/issue16105_v4.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16105

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-04 Thread Felipe Cruz
Felipe Cruz added the comment: This patch retries write() until success if errno == EINTR. There is also a test. -- Added file: http://bugs.python.org/file27428/issue16105_v3.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-03 Thread Felipe Cruz
Felipe Cruz added the comment: Why limit to EBADF? You could also have EPIPE, EINVAL and many other errors. The only error you may not want to report is EAGAIN. Charles, You're right! If all errno cases get covered in the patch, will It looks reasonable

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-03 Thread Felipe Cruz
Felipe Cruz added the comment: Raising an error in case the signal can't be written to the FD (because the other end didn't drain the pipe/socket) seems reasonable. You should just retry on EINTR (although any sane implementation shouldn't return EINTR on non-blocking write, I don't think

[issue15897] zipimport.c doesn't check return value of fseek()

2012-10-02 Thread Felipe Cruz
Felipe Cruz added the comment: Should I send patches for 3.2 and 2.7? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15897 ___ ___ Python-bugs

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-02 Thread Felipe Cruz
Felipe Cruz added the comment: Hello, since Antonie mentioned Py_AddPendingCall I came up with a patch describing what he proposed. Let me know if this patch can be improved or discarded(if the problem requires a more sophisticated solution). In case of improvement I can also submit another

[issue15897] zipimport.c doesn't check return value of fseek()

2012-10-01 Thread Felipe Cruz
Felipe Cruz added the comment: Hello! Just sent the Contributor Agreement Form. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15897

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-01 Thread Felipe Cruz
New submission from Felipe Cruz: It's possible to set a read only FD to signal.set_wakeup_fd(fd) Since write call[1] inside 'trip_signal' return code is ignored, no error will be raised. An untested solution is to call fcntl in this FD to check presence of write flags. 1 - http

[issue16105] Pass read only FD to signal.set_wakeup_fd

2012-10-01 Thread Felipe Cruz
Felipe Cruz added the comment: I would not say that is a bug, but there is a write(wakeup_fd) call with ignored return code and maybe this can be improved to an output to stderr, or maybe a better solution. -- ___ Python tracker rep

[issue15744] missing tests for {RawIO,BufferedIO,TextIO}.writelines

2012-09-18 Thread Felipe Cruz
Felipe Cruz added the comment: Updated based on Pitrou comments -- Added file: http://bugs.python.org/file27220/issue15744_v2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15744

[issue15897] zipimport.c doesn't check return value of fseek()

2012-09-15 Thread Felipe Cruz
Felipe Cruz added the comment: I've updated the patch changing fseek_error goto block error return from PyErr_SetFromErrno to PyErr_Format(ZipImportError, can't read Zip file: %R, archive); (returning NULL after). -- Added file: http://bugs.python.org/file27194/issue15897_v1.patch

[issue15897] zipimport.c doesn't check return value of fseek()

2012-09-15 Thread Felipe Cruz
Felipe Cruz added the comment: v4 - inline fseek return code check - as Christian suggested -- Added file: http://bugs.python.org/file27195/issue15897_v4.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15897

[issue15948] Unchecked return value of I/O functions

2012-09-15 Thread Felipe Cruz
Felipe Cruz added the comment: I can submit patches.. Is there any problem to send 1 patch per module? -- nosy: +felipecruz ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue15948

[issue15744] missing tests for {RawIO,BufferedIO,TextIO}.writelines

2012-09-15 Thread Felipe Cruz
Felipe Cruz added the comment: Add tests for {RawIO,BufferedIO,TextIO}.writelines() -- keywords: +patch nosy: +felipecruz Added file: http://bugs.python.org/file27200/issue15744_v1.patch ___ Python tracker rep...@bugs.python.org http

[issue15897] zipimport.c doesn't check return value of fseek()

2012-09-14 Thread Felipe Cruz
Felipe Cruz added the comment: Hello! This is one of my first patches. Tests still OK! Let me know what you think! Thanks! -- keywords: +patch nosy: +felipecruz Added file: http://bugs.python.org/file27191/issue15897_v1.patch ___ Python tracker

[issue15897] zipimport.c doesn't check return value of fseek()

2012-09-14 Thread Felipe Cruz
Felipe Cruz added the comment: Patch updated - fseek errors messges will be can't read Zip file' and not can't Open Zip file -- Added file: http://bugs.python.org/file27192/issue15897_v1.patch ___ Python tracker rep...@bugs.python.org http

[issue7484] smtplib: verify breaks with Postfix servers

2011-07-18 Thread Felipe Cruz
Felipe Cruz felipec...@loogica.net added the comment: You're very kind David. Hope I can contribute with something more relevant next time :) best regards, Felipe 2011/7/18 R. David Murray rep...@bugs.python.org R. David Murray rdmur...@bitdance.com added the comment: Thank you both

[issue7484] smtplib: verify breaks with Postfix servers

2011-07-13 Thread Felipe Cruz
Felipe Cruz felipec...@loogica.net added the comment: Can anyone take a loot at those patches? Do they need more tests? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7484

[issue7484] smtplib: verify breaks with Postfix servers

2011-04-13 Thread Felipe Cruz
Felipe Cruz felipec...@loogica.net added the comment: David.. I extracted quoteaddr code to _addrformat and now quoteaddr and _addronly call _addrformat passing a format (%s or %s). I've also created quoteaddr and _addronly test functions as well modified VRFY and EXPN tests to make sure

[issue7484] smtplib: verify breaks with Postfix servers

2011-04-13 Thread Felipe Cruz
Changes by Felipe Cruz felipec...@loogica.net: Added file: http://bugs.python.org/file21652/issue7484-27_2.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7484

[issue7484] smtplib: verify breaks with Postfix servers

2011-04-12 Thread Felipe Cruz
Felipe Cruz felipec...@loogica.net added the comment: I've rewrote those patches to 'default' and 2.7 -- nosy: +felipecruz Added file: http://bugs.python.org/file21641/issue7484-py3k.diff ___ Python tracker rep...@bugs.python.org http

[issue7484] smtplib: verify breaks with Postfix servers

2011-04-12 Thread Felipe Cruz
Changes by Felipe Cruz felipec...@loogica.net: Added file: http://bugs.python.org/file21642/issue7484-27.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue7484

[issue3410] platform.version() don't work as expected in Vista in portuguese

2009-07-06 Thread Felipe Portella
Felipe Portella fel...@portella.com.br added the comment: The same happens in Portuguese version ... the regex fails because ver returns Versão ... []'s On Mon, Jul 6, 2009 at 7:54 AM, Ezio Melotti rep...@bugs.python.org wrote: Ezio Melotti ezio.melo...@gmail.com added the comment: Won't

[issue4164] String double quoted fatal problem

2008-10-21 Thread Felipe
New submission from Felipe [EMAIL PROTECTED]: Hi I have a problem with python 2.6, when i try to process strings with triple-quoted string literal i get an error: a='a''a' #1 double quoted Ok a='aa' # 2 Ok a= 'a''a' # 3 doble quoted -- SyntaxError: EOF while scanning triple-quoted

[issue3410] platform.version() don't work as expected in Vista in portuguese

2008-07-18 Thread Felipe Portella
New submission from Felipe Portella [EMAIL PROTECTED]: Using Vista in Portuguese platform.version is returning 32bits instead of 6.0.6001. This is because in file platform.py line 379 thee regular expression try to search for the word Version in english, while in Portuguese the command ver