From: Roland Haeder <roland@mxchange.org>
Date: Fri, 17 Oct 2014 22:34:55 +0000 (+0200)
Subject: Double qoutes generally "cost" performance as they are searched for variables (and... 
X-Git-Url: https://git.mxchange.org/?p=core.git;a=commitdiff_plain;h=b7a1150d7fe85bab5d50577bef96c7e708e34dbf;hp=8dfb880fec40772559887963d68e380924ca1061;ds=sidebyside

Double qoutes generally "cost" performance as they are searched for variables (and single quotes not).

Signed-off-by: Roland Häder <roland@mxchange.org>
---

diff --git a/inc/classes/main/helper/class_BaseHelper.php b/inc/classes/main/helper/class_BaseHelper.php
index 8616a502..73643974 100644
--- a/inc/classes/main/helper/class_BaseHelper.php
+++ b/inc/classes/main/helper/class_BaseHelper.php
@@ -253,7 +253,7 @@ class BaseHelper extends BaseFrameworkSystem {
 		$this->groups[$this->totalCounter] = $groupId;
 		$this->groups[$groupId]['opened']  = TRUE;
 		$this->groups[$groupId]['content'] = sprintf(
-			"<!-- group %s opened (length: %s, tag: %s) //-->%s\n",
+			'<!-- group %s opened (length: %s, tag: %s) //-->%s' . PHP_EOL,
 			$groupId,
 			strlen($content),
 			$tag,
@@ -367,11 +367,11 @@ class BaseHelper extends BaseFrameworkSystem {
 		// Is the content empty?
 		if ((empty($content)) && (!empty($this->subGroups[$subGroupId]['tag']))) {
 			// Get it from opener
-			$content = sprintf("<!-- sub-group %s auto-closed //--></%s>", $subGroupId, $this->subGroups[$subGroupId]['tag']);
+			$content = sprintf('<!-- sub-group %s auto-closed //--></%s>', $subGroupId, $this->subGroups[$subGroupId]['tag']);
 		} // END - if
 
 		// Add content to it and mark it as closed
-		$this->subGroups[$subGroupId]['content'] .= sprintf("<!-- sub-group %s closed (length: %s, tag: %s) //-->%s\n", $subGroupId, strlen($content), $this->subGroups[$subGroupId]['tag'], $content);
+		$this->subGroups[$subGroupId]['content'] .= sprintf('<!-- sub-group %s closed (length: %s, tag: %s) //-->%s' . PHP_EOL, $subGroupId, strlen($content), $this->subGroups[$subGroupId]['tag'], $content);
 		$this->subGroups[$subGroupId]['opened'] = FALSE
 		;
 
@@ -413,7 +413,7 @@ class BaseHelper extends BaseFrameworkSystem {
 				$content .= trim($subGroupContent);
 			} else {
 				// Something went wrong
-				$this->debugInstance(__METHOD__."(): Something unexpected happened here.");
+				$this->debugInstance(__METHOD__ . '(): Something unexpected happened here.');
 			}
 		} // END - for