[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-10-19 Thread Roundup Robot
Roundup Robot added the comment: New changeset f35514dfadf8 by Senthil Kumaran in branch '2.7': Fix Issue 12604 - Use a proper no-op macro expansion for VTRACE macro in _sre.c http://hg.python.org/cpython/rev/f35514dfadf8 New changeset ab028084f704 by Senthil Kumaran in branch '3.2': 3.2 - Fix

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-08-08 Thread Petri Lehtinen
Petri Lehtinen added the comment: Éric Araujo wrote: > _sre is a module written in C, not Python; we call that an extension > module. The Library component is used for Python modules, which are > found in the Lib directory in a checkout of CPython. See #12711. Ok, I wasn't aware of this. Thanks

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-08-08 Thread Éric Araujo
Changes by Éric Araujo : -- stage: patch review -> commit review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-08-08 Thread Éric Araujo
Éric Araujo added the comment: _sre is a module written in C, not Python; we call that an extension module. The Library component is used for Python modules, which are found in the Lib directory in a checkout of CPython. See #12711. -- nosy: +eric.araujo ___

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-07-23 Thread Petri Lehtinen
Petri Lehtinen added the comment: Should the component really be extension modules? I don't believe _sre.c is a part of the API, and the problem is embedding Python as a whole. -- ___ Python tracker _

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-07-22 Thread Éric Araujo
Changes by Éric Araujo : -- components: +Extension Modules -Library (Lib) versions: +Python 3.2, Python 3.3 ___ Python tracker ___ ___

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-07-22 Thread Petri Lehtinen
Changes by Petri Lehtinen : -- components: +Build, Library (Lib) keywords: +needs review stage: -> patch review ___ Python tracker ___ __

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-07-22 Thread Petri Lehtinen
Petri Lehtinen added the comment: Attached a patch against 2.7 that adds the suggested fix. -- keywords: +patch nosy: +petri.lehtinen Added file: http://bugs.python.org/file22718/fix_empty_macro.patch ___ Python tracker

[issue12604] VTRACE macro in _sre.c should use do {} while (0)

2011-07-21 Thread Josh Triplett
New submission from Josh Triplett : In _sre.c, the VTRACE macro normally gets defined to nothing. It later gets used as the body of control structures such as "else" without braces, which causes many compilers to warn (to catch stray semicolons like "else;"). This makes it difficult to compi