Saved four invocations to strpos() in sqimap_find_displayable_name()
authorantipode <antipode@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Fri, 28 Dec 2001 18:47:42 +0000 (18:47 +0000)
committerantipode <antipode@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Fri, 28 Dec 2001 18:47:42 +0000 (18:47 +0000)
(take 2) :^)

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@1989 7612ce4b-ef26-0410-bec9-ea0150e637f0

functions/imap_general.php

index a9e1882..e07bbc9 100755 (executable)
@@ -343,23 +343,21 @@ function sqimap_find_email ($string) {
 function sqimap_find_displayable_name ($string) {
     $string = ' '.trim($string);
     $orig_string = $string;
-    if (strpos($string, '<') && strpos($string, '>')) {
-        if (strpos($string, '<') == 1) {
+    if (($angle1 = strpos($string, '<')) && strpos($string, '>')) {
+        if ($angle1 == 1) {
             $string = sqimap_find_email($string);
         } else {
             $string = trim($string);
-            $string = substr($string, 0, strpos($string, '<'));
+            $string = substr($string, 0, $angle1-1);
             $string = ereg_replace ('"', '', $string);
         }
         
         if (trim($string) == '') {
             $string = sqimap_find_email($orig_string);
         }
-     } else if (strpos($string, '(') && strpos($string, ')')) {
-         $fn_start = strpos($string, '(') + 1;
-         $fn_len   = strpos($string, ')') - $fn_start;
-         $string = substr($string, $fn_start, $fn_len);
+    } else if ( ($paren1 = strpos($string, '('))
+                && ($paren2 = strpos($string, ')'))) {
+        $string = substr($string, $paren1 + 1, $paren2 - $paren1 - 1);
     }
     return $string;
 }