commit 32ef88bdd04a7ec83394905746e47c77fba2627b
author Shih-wei Liao <sliao@google.com> Sat Feb 04 23:33:11 2012 -0800
committer Shih-wei Liao <sliao@google.com> Sat Feb 04 23:33:29 2012 -0800
tree 760d9d1a1ccbd38c6459131039169a1ccc54fcbd
parent 743dd7144d2f9f60cc9000cdb1cab490846136a7

Minor refactoring.

Change-Id: I70123dd22672dacd30d7f7624af8d82d8a0e8525

lib/ExecutionEngine/Script.cpp

diff --git a/lib/ExecutionEngine/Script.cpp b/lib/ExecutionEngine/Script.cpp
index 024ac7e..1540967 100644
--- a/lib/ExecutionEngine/Script.cpp
+++ b/lib/ExecutionEngine/Script.cpp
@@ -186,8 +186,9 @@
                                unsigned long flags) {
 #if USE_CACHE
   if (internalLoadCache(cacheDir, cacheName,
-                        ScriptObject::Relocatable, /* checkOnly */true) == 0)
+                        ScriptObject::Relocatable, /* checkOnly */ true) == 0) {
     return 0;
+  }
 #endif
 
   CompilerOption option;
@@ -212,8 +213,9 @@
 
 #if USE_CACHE
   if (internalLoadCache(cacheDir, cacheName,
-                        ScriptObject::Executable, /* checkOnly */false) == 0)
+                        ScriptObject::Executable, /* checkOnly */ false) == 0) {
     return 0;
+  }
 #endif
 
   CompilerOption option;
@@ -231,19 +233,22 @@
                               bool checkOnly) {
   mObjectType = objectType;
 
-  if ((cacheDir == NULL) || (cacheName == NULL))
+  if ((cacheDir == NULL) || (cacheName == NULL)) {
     return 1;
+  }
 
   // Set cache file Name
   mCacheName = cacheName;
 
-  // Santize the mCacheDir. Ensure that mCacheDir is end with '/'.
+  // Santize mCacheDir. Ensure that mCacheDir ends with '/'.
   mCacheDir = cacheDir;
-  if (!mCacheDir.empty() && *mCacheDir.rbegin() != '/')
+  if (!mCacheDir.empty() && *mCacheDir.rbegin() != '/') {
     mCacheDir.push_back('/');
+  }
 
-  if (!isCacheable())
+  if (!isCacheable()) {
     return 1;
+  }
 
   std::string objPath = getCachedObjectPath();
   std::string infoPath = getCacheInfoPath();