Message168625
| Author |
ag6502 |
| Recipients |
Mark.Shannon, ag6502, amaury.forgeotdarc, benjamin.peterson, francismb, ggenellina, pitrou, serhiy.storchaka, terry.reedy |
| Date |
2012年08月20日.06:53:59 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1345445662.93.0.239125199081.issue5765@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I missed all the macrology present :-( ... the following is a patch that takes it into account (also defines a VISIT_QUIT macro to make more visible the exit points). The handling has been also extended to visit_stmt because the macros are shared.
Of course all this makes sense assuming that a cleanup in case of error is indeed desired...
BTW: shouldn't be all those statement macros of the "do{...}while(0)" form instead of just being wrapped in "{}" ? I see potential problems with if/else... |
|