Message255245
The patch looks technically good, though I’m not a big fan of the double underscores, like element_get__text(). Maybe element_text_getter() instead, because it has to match the “getter” signature?
Personally, I would say keep the doc strings basic. If you want to add more depth about “tail”, I suggest to keep it to the main RST documentation (which was updated fairly recently). But it would be good to quickly distinguish text and tail other than by name. Maybe:
{"text", ..., "A string of text directly after the start tag, or None"},
{"tail", ..., "A string of text directly after the end tag, or None"}, |
|
Date |
User |
Action |
Args |
2015-11-24 04:04:50 | martin.panter | set | recipients:
+ martin.panter, effbot, christian.heimes, eli.bendersky, ArcRiley, flox, santoso.wijaya, serhiy.storchaka |
2015-11-24 04:04:50 | martin.panter | set | messageid: <1448337890.78.0.473910088338.issue7990@psf.upfronthosting.co.za> |
2015-11-24 04:04:50 | martin.panter | link | issue7990 messages |
2015-11-24 04:04:50 | martin.panter | create | |
|