Message 280112 - Python tracker

➜

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.

In-reply-to
Author	xiang.zhang
Recipients	Carl Ekerot, loewis, serhiy.storchaka, xiang.zhang
Date	2016年11月05日.13:36:53
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1478353013.63.0.505798838823.issue28563@psf.upfronthosting.co.za>

Content
> '1?2:3?4:5' -> '(2 if 1 else 3)?4:5' -> '(4 if (2 if 1 else 3) else 5' This is not right. It's right associative so it should be 1?2:(3?4:5) -> 1?2:(4 if 3 else 5) -> 2 if 1 else (4 if 3 else 5) > It would be nice to make c2py() working with any expressions, but if this is too hard, this can be left for other issue. Agree. But I am interested in trying. > gettext_c2py.patch itself LGTM for fixing security issue, but tests are needed. It gets drawbacks so I don't include tests. I'll add in next try.

Content

> '1?2:3?4:5' -> '(2 if 1 else 3)?4:5' -> '(4 if (2 if 1 else 3) else 5'
This is not right. It's right associative so it should be
1?2:(3?4:5) -> 1?2:(4 if 3 else 5) -> 2 if 1 else (4 if 3 else 5)
> It would be nice to make c2py() working with any expressions, but if this is too hard, this can be left for other issue.
Agree. But I am interested in trying.
> gettext_c2py.patch itself LGTM for fixing security issue, but tests are needed.
It gets drawbacks so I don't include tests. I'll add in next try.

History
Date	User	Action	Args
2016年11月05日 13:36:53	xiang.zhang	set	recipients: + xiang.zhang, loewis, serhiy.storchaka, Carl Ekerot
2016年11月05日 13:36:53	xiang.zhang	set	messageid: <1478353013.63.0.505798838823.issue28563@psf.upfronthosting.co.za>
2016年11月05日 13:36:53	xiang.zhang	link	issue28563 messages
2016年11月05日 13:36:53	xiang.zhang	create

homepage