Make all linters happy.

opal · Dec 30, 2018 · a1c0503 · a1c0503
1 parent 32927b5
commit a1c0503
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/lib/opal/nodes/base.rb b/lib/opal/nodes/base.rb
@@ -169,7 +169,7 @@ def class_variable_owner_nesting_level
         cvar_scope = scope
         nesting_level = 0
 
-        while cvar_scope && !(cvar_scope.class_scope?)
+        while cvar_scope && !cvar_scope.class_scope?
           # Needs only `class << self`, `module`, and `class`
           # can increase nesting, but `class` & `module` are
           # covered by `class_scope?`.

diff --git a/opal/corelib/runtime.js b/opal/corelib/runtime.js
@@ -400,7 +400,7 @@
   // @return new [Class]  or existing ruby class
   //
   Opal.allocate_class = function(name, superclass) {
-    var constructor;
+    var klass, constructor;
 
     if (superclass != null && superclass.$$bridge) {
       // Inheritance from bridged classes requires
@@ -417,10 +417,11 @@
       constructor = function(){};
     }
 
-    if (name)
+    if (name) {
       $defineProperty(constructor, 'displayName', '::'+name);
+    }
 
-    var klass = constructor;
+    klass = constructor;
 
     $defineProperty(klass, '$$name', name);
     $defineProperty(klass, '$$constructor', constructor);
@@ -550,8 +551,9 @@
   // @return [Module]
   Opal.allocate_module = function(name) {
     var constructor = function(){};
-    if (name)
+    if (name) {
       $defineProperty(constructor, 'displayName', name+'.$$constructor');
+    }
 
     var module = constructor;
 
@@ -606,7 +608,7 @@
     }
 
     // Module doesnt exist, create a new one...
-    module = Opal.allocate_module(name, constructor);
+    module = Opal.allocate_module(name);
     Opal.const_set(scope, name, module);
 
     return module;