Message147686
| Author |
eric.araujo |
| Recipients |
alex, cvrebert, eric.araujo, ncoghlan, pitrou |
| Date |
2011年11月15日.16:11:02 |
| SpamBayes Score |
2.1124772e-06 |
| Marked as misclassified |
No |
| Message-id |
<1321373463.47.0.234808535923.issue13238@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
> def __init__(self, command, *, **callkwds):
Is the '*' marker needed?
> self.callkwds = callkwds
These aren’t used in the module-level functions. What is the use case?
If you forgive me for the nitpick, the docstrings have too much indenting.
> a "*.py" value interpolated with "{!u}" should indeed pick up all of those files,
> since the wildcard will be passed unmodified to the underlying shell
Great.
(This patch will also serve as a nice example of creating a string formatter. I’ve seen the PyCon video about them, but I thought I was missing a piece since I didn’t see how you hook the format function to your custom formatter; it looks like the answer is that you don’t.) |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2011年11月15日 16:11:03 | eric.araujo | set | recipients:
+ eric.araujo, ncoghlan, pitrou, alex, cvrebert |
| 2011年11月15日 16:11:03 | eric.araujo | set | messageid: <1321373463.47.0.234808535923.issue13238@psf.upfronthosting.co.za> |
| 2011年11月15日 16:11:02 | eric.araujo | link | issue13238 messages |
| 2011年11月15日 16:11:02 | eric.araujo | create |
|