summaryrefslogtreecommitdiff
path: root/pp_ctl.c
diff options
context:
space:
mode:
authorDavid Mitchell <davem@iabyn.com>2017-02-18 14:00:56 +0000
committerDavid Mitchell <davem@iabyn.com>2017-02-18 14:00:56 +0000
commit3b1d752dd57309cbf5f5018ddf5ec58ce2fcb7a4 (patch)
tree605a6bf9ad5cf618a821919e34bd2143741de1b5 /pp_ctl.c
parentac7609e4a8cf90d4a873cf5753face7d2c5f3976 (diff)
downloadperl-3b1d752dd57309cbf5f5018ddf5ec58ce2fcb7a4.tar.gz
pp_formline(): add empty body to empty while loop
my previous commit in this function added a block that happened to follow directly after a bodiless while loop, i.e. 'while(...);'. clang spotted this and warned. So add an empty body '{}' after the while to visually disambiguate it.
Diffstat (limited to 'pp_ctl.c')
-rw-r--r--pp_ctl.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/pp_ctl.c b/pp_ctl.c
index c4c8bcd0c1..44498bc3fe 100644
--- a/pp_ctl.c
+++ b/pp_ctl.c
@@ -878,7 +878,9 @@ PP(pp_formline)
case FF_NEWLINE: /* delete trailing spaces, then append \n */
f++;
- while (t-- > (SvPVX(PL_formtarget) + linemark) && *t == ' ') ;
+ while (t-- > (SvPVX(PL_formtarget) + linemark) && *t == ' ')
+ {}
+
{
STRLEN cur = t - SvPVX_const(PL_formtarget);
t = SvGROW(PL_formtarget, cur + 1 + 1) + cur;