From: stekkel <stekkel@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Wed, 16 Jun 2004 09:48:31 +0000 (+0000)
Subject: Fixed warning.
X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=commitdiff_plain;h=be86a35a18a97ea0e1bb16890d5737dd6ec73343;hp=bb9773940a9f9f8f13922c065f591d1d7379328e;ds=sidebyside

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
---

diff --git a/functions/strings.php b/functions/strings.php
index 924f2f1b..242a29b9 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} == '-' ||