Changeset 5135 in subversion

Timestamp:

Aug 26, 2011 5:22:53 AM (21 months ago)

Author:

alec

Message:

• Microoptimization: use substr_replace() for injecting a string into string

File:

• trunk/roundcubemail/program/include/rcube_html_page.php (modified) (5 diffs)

trunk/roundcubemail/program/include/rcube_html_page.php

@@ -213 +213 @@
             $this->add_script('$(document).ready(function(){ ' . $this->scripts['docready'] . "\n});", 'foot');
         }
-        
+
         if (is_array($this->script_files['foot'])) {
             foreach ($this->script_files['foot'] as $file) {
@@ -247 +247 @@
         // add page hader
         if ($hpos) {
-            $output = substr($output,0,$hpos) . $page_header . substr($output,$hpos,strlen($output));
+            $output = substr_replace($output, $page_header, $hpos, 0);
         }
         else {
@@ -255 +255 @@
         // add page footer
         if (($fpos = strripos($output, '</body>')) || ($fpos = strripos($output, '</html>'))) {
-            $output = substr($output, 0, $fpos) . "$page_footer\n" . substr($output, $fpos);
+            $output = substr_replace($output, $page_footer."\n", $fpos, 0);
         }
         else {
@@ -269 +269 @@
                 $css .= sprintf($this->link_css_file, $file);
             }
-            $output = substr($output, 0, $pos) . $css . substr($output, $pos);
+            $output = substr_replace($output, $css, $pos, 0);
         }
 
@@ -290 +290 @@
         }
     }
-    
+
     /**
      * Callback function for preg_replace_callback in write()