Message253364
Okay, sorry for repeated nitpicks, but I think you can keep the visibility of sigwinch_received as static like it was as char (to avoid having it be exported for potential use by other modules, though it won't gain any optimization benefits of static, since volatile intentionally prevents any such optimizations). static volatile sig_atomic_t is kind of a mouthful, but keeps the limited visibility you want.
Also, I still don't see a review link for the patch (which limits the ability to review in context); you might need to update your checkout and regenerate the patch so the review tool can handle it. |
|
Date |
User |
Action |
Args |
2015-10-23 03:09:21 | josh.r | set | recipients:
+ josh.r, r.david.murray, neologix, takluyver, martin.panter, Carlos Pita, johnmorr, Eric Price |
2015-10-23 03:09:21 | josh.r | set | messageid: <1445569761.33.0.289977805962.issue23735@psf.upfronthosting.co.za> |
2015-10-23 03:09:21 | josh.r | link | issue23735 messages |
2015-10-23 03:09:20 | josh.r | create | |
|