Improve clarity of help format_line() splitting.

Thanks to Zero King for the initial patch.  He says the original line
confused some static analysis tools.

I don't see anything wrong with the operator precedence, but as long
as we are cleaning up the line, split assignment and conditional
evaluation to make it even clearer.

1 files changed, 3 insertions, 3 deletions

diff --git a/help.c b/help.c
index c5bb4b7f..cda1b996 100644
--- a/help.c
+++ b/help.c
@@ -207,9 +207,9 @@ static void format_line (FILE *f, int ismacro,
 
   fputs (t1, f);
 
-  /* don't try to press string into one line with less than 40 characters.
-     The double parenthesis avoids a gcc warning, sigh ... */
-  if ((split = MuttIndexWindow->cols < 40))
+  /* don't try to press string into one line with less than 40 characters. */
+  split = (MuttIndexWindow->cols < 40);
+  if (split)
   {
     col_a = col = 0;
     col_b = LONG_STRING;