This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eli.bendersky
Recipients amaury.forgeotdarc, belopolsky, eli.bendersky, ezio.melotti, georg.brandl, pitrou, terry.reedy
Date 2010-08-07.06:20:46
SpamBayes Score 5.0351695e-10
Marked as misclassified No
Message-id <1281162054.28.0.885866668639.issue9315@psf.upfronthosting.co.za>
In-reply-to
Content
[attaching a new patch version]

1. Are you refering to Lib/test/tracedmodules/__init__.py? I did add it - it appears in the patchfile. What do you mean?

2. Fixed

3. Fixed

4. I'm not sure. Tracing of the current file is also a common use case, and I really only created the tracedmodules package to test tracing throughout module boundaries. Anyway I don't have a strong opinion on this issue, so if you think it's important I can move them.

5. 8. I added a comment above the traced functions. For (8) moved the content of README into __init__.py, which documents this re the tracedmodules files in the tracedmodules/ dir.

6. I've documented this in the __init__.py file, once and globally for all traced modules as I don't want to repeat the same comment for each file that may be created there in the future. If you think this isn't enough, please advise re the comment you'd like to see in there.

7. Fixed

9. Modified the function's name to fix_ext_py and its functionality to what trace and some other modules do. Removed the 'modname' function and folded its functionality into my_file_and_modname. Re the latter, I prefer to leave it as a function because it's more flexible this way (flexible for changes in the trace format, for example), and also I almost always use the pair as a tuple anyway, so the function's returning the tuple is more convenient than always pairing two long constant names. The cost of recomputation is meaningless in the context of this code.

10. Fixed

11. Fixed
History
Date User Action Args
2010-08-07 06:20:54eli.benderskysetrecipients: + eli.bendersky, georg.brandl, terry.reedy, amaury.forgeotdarc, belopolsky, pitrou, ezio.melotti
2010-08-07 06:20:54eli.benderskysetmessageid: <1281162054.28.0.885866668639.issue9315@psf.upfronthosting.co.za>
2010-08-07 06:20:52eli.benderskylinkissue9315 messages
2010-08-07 06:20:50eli.benderskycreate