From be86a35a18a97ea0e1bb16890d5737dd6ec73343 Mon Sep 17 00:00:00 2001
From: stekkel <stekkel@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Wed, 16 Jun 2004 09:48:31 +0000
Subject: [PATCH] Fixed warning.

TODO: look at lines which are longer then the editor size and and theirfor
do not reach the code responsable for the smart wrapping of short lines.
Those lines should also be tested on first words.


git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@7665 7612ce4b-ef26-0410-bec9-ea0150e637f0
---
 functions/strings.php | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/functions/strings.php b/functions/strings.php
index 924f2f1..242a29b 100644
--- a/functions/strings.php
+++ b/functions/strings.php
@@ -213,7 +213,6 @@ function &sqBodyWrap (&$body, $wrap) {
         * Set this to false to stop appending short strings to previous lines
         */
        $smartwrap = true;
-
        // inner loop, (obviously) handles wrapping up to
        // the next newline
        while ($pos < $nextNewline) {
@@ -221,8 +220,6 @@ function &sqBodyWrap (&$body, $wrap) {
            while (($pos < $nextNewline) && (ctype_space ($body{$pos}))) {
                $pos++;
            }
-
-
            // if this is a short line then just append it and continue outer loop
            if (($outStringCol + $nextNewline - $pos) <= ($wrap - $citeLevel - 1) ) {
                // if this is the final line in the input string then include
@@ -237,16 +234,15 @@ function &sqBodyWrap (&$body, $wrap) {
                while (($lastRealChar > $pos && $lastRealChar < $length) && (ctype_space ($body{$lastRealChar}))) {
                    $lastRealChar--;
                }
-
                // decide if appending the short string is what we want
                if (($nextNewline < $length && $body{$nextNewline} == "\n") &&
                      isset($lastRealChar)) {
-
-                     //check the first word:
-                   $mypos = $nextNewline+1;
+                   $mypos = $pos;
+                   //check the first word:
                    while (($mypos < $length) && ($body{$mypos} == '>')) {
                        $mypos++;
                        // skip over any spaces interleaved among the cite markers
+                       $oldpos = $mypos;
                        while (($mypos < $length) && ($body{$mypos} == ' ')) {
                            $mypos++;
                        }
@@ -261,7 +257,6 @@ function &sqBodyWrap (&$body, $wrap) {
 */
 
                    $firstword = substr($body,$mypos,strpos($body,' ',$mypos) - $mypos);
-
                    //if ($dowrap || $ldnspacecnt > 1 || ($firstword && (
                    if (!$smartwrap || $firstword && (
                                         $firstword{0} == '-' ||
-- 
1.9.1