opal 1.3.2 → 1.4.0.alpha1

data/spec/filters/unsupported/kernel.rb

@@ -19,6 +19,8 @@ opal_unsupported_filter "Kernel" do
   fails "Kernel#eql? is a public instance method"
   fails "Kernel#fail is a private method"
   fails "Kernel#format is a private method"
+  fails "Kernel#initialize_clone returns the receiver" # Expected nil == #<Object:0x63440> to be truthy but was false
+  fails "Kernel#initialize_dup returns the receiver" # Expected nil == #<Object:0x4c314> to be truthy but was false
   fails "Kernel#inspect returns an untrusted string if self is untrusted"
   fails "Kernel#raise is a private method"
   fails "Kernel#sleep accepts a Rational"
@@ -27,9 +29,19 @@ opal_unsupported_filter "Kernel" do
   fails "Kernel#sprintf is a private method"
   fails "Kernel#to_s returns a tainted result if self is tainted"
   fails "Kernel#to_s returns an untrusted result if self is untrusted"
+  fails "Kernel#warn calls Warning.warn with category: nil if Warning.warn accepts keyword arguments" # NameError: uninitialized constant Warning
+  fails "Kernel#warn calls Warning.warn with given category keyword converted to a symbol" # NameError: uninitialized constant Warning
+  fails "Kernel#warn calls Warning.warn without keyword arguments if Warning.warn does not accept keyword arguments" # NameError: uninitialized constant Warning
   fails "Kernel#warn is a private method"
   fails "Kernel.Float raises a TypeError if #to_f returns an Integer"
   fails "Kernel.Integer calls to_i on Rationals"
   fails "Kernel.Integer returns a Fixnum or Bignum object"
   fails "Kernel.fail is a private method"
+  fails "Kernel.lambda when called without a literal block warns when proc isn't a lambda" # Expected warning: "ruby/core/kernel/lambda_spec.rb:142: warning: lambda without a literal block is deprecated; use the proc without lambda instead\n" but got: ""
+  fails "Kernel.printf formatting io is specified other formats s formats nil with width and precision" # Exception: format_string.indexOf is not a function
+  fails "Kernel.printf formatting io is specified other formats s formats nli with precision" # Exception: format_string.indexOf is not a function
+  fails "Kernel.printf formatting io is specified other formats s formats nli with width" # Exception: format_string.indexOf is not a function
+  fails "Kernel.printf formatting io is specified other formats s formats string with width and precision" # Exception: format_string.indexOf is not a function
+  fails "Kernel.printf formatting io is specified other formats s formats string with width" # Exception: format_string.indexOf is not a function
+  fails "Kernel.printf formatting io is specified other formats s substitutes '' for nil" # Exception: format_string.indexOf is not a function
 end

data/spec/filters/unsupported/privacy.rb

@@ -15,12 +15,15 @@ opal_unsupported_filter "private" do
   fails "Defining an 'initialize_copy' method sets the method's visibility to private"
   fails "Defining an 'initialize_dup' method sets the method's visibility to private"
   fails "Enumerator#initialize is a private method"
+  fails "Enumerator::Chain#initialize is a private method" # Expected Enumerator::Chain to have private instance method 'initialize' but it does not
   fails "Enumerator::Generator#initialize is a private method" # Expected Enumerator::Generator to have private instance method 'initialize' but it does not
   fails "Enumerator::Lazy#initialize is a private method" # Expected Enumerator::Lazy to have private instance method 'initialize' but it does not
   fails "Enumerator::Yielder#initialize is a private method" # Expected Enumerator::Yielder to have private instance method 'initialize' but it does not
   fails "Invoking a private getter method does not permit self as a receiver"
   fails "Kernel#block_given? is a private method"
   fails "Kernel#eval is a private method"
+  fails "Kernel#initialize_clone is a private instance method" # Expected Kernel to have private instance method 'initialize_clone' but it does not
+  fails "Kernel#initialize_dup is a private instance method" # Expected Kernel to have private instance method 'initialize_dup' but it does not
   fails "Kernel#iterator? is a private method"
   fails "Kernel#local_variables is a private method"
   fails "Kernel#methods returns private singleton methods defined by obj.meth"

data/spec/filters/unsupported/string.rb

@@ -419,6 +419,7 @@ opal_unsupported_filter "String" do
   fails "String#lstrip! raises a RuntimeError on a frozen instance that is modified"
   fails "String#lstrip! raises a RuntimeError on a frozen instance that would not be modified"
   fails "String#lstrip! returns nil if no modifications were made"
+  fails "String#lstrip! strips leading \\0" # NotImplementedError: String#lstrip! not supported. Mutable String methods are not supported in Opal.
   fails "String#match matches \\G at the start of the string"
   fails "String#next taints the result if self is tainted"
   fails "String#next! is equivalent to succ, but modifies self in place (still returns self)"
@@ -554,6 +555,7 @@ opal_unsupported_filter "String" do
   fails "String#strip! modifies self removing trailing NULL bytes and whitespace"
   fails "String#strip! raises a RuntimeError on a frozen instance that is modified"
   fails "String#strip! raises a RuntimeError on a frozen instance that would not be modified"
+  fails "String#strip! removes leading and trailing NULL bytes and whitespace" # NotImplementedError: String#strip! not supported. Mutable String methods are not supported in Opal.
   fails "String#strip! returns nil if no modifications where made"
   fails "String#sub with pattern and Hash ignores non-String keys" # Expected "tazoo" == "taboo" to be truthy but was false
   fails "String#sub with pattern and Hash taints the result if a hash value is tainted"

data/spec/lib/builder_spec.rb

@@ -76,14 +76,14 @@ RSpec.describe Opal::Builder do
     expect(Opal::Config.compiler_options[:arity_check]).to eq(false)
     builder = described_class.new
     builder.build_str('def foo; end', 'foo')
-    expect(builder.to_s).not_to include('$foo$1.$$parameters = []')
+    expect(builder.to_s).not_to include('$$parameters: []')
 
     Opal::Config.arity_check_enabled = true
     expect(Opal::Config.arity_check_enabled).to eq(true)
     expect(Opal::Config.compiler_options[:arity_check]).to eq(true)
     builder = described_class.new
     builder.build_str('def foo; end', 'foo')
-    expect(builder.to_s).to include('$foo$1.$$parameters = []')
+    expect(builder.to_s).to include('$$parameters: []')
   end
 
   describe '#missing_require_severity' do

data/spec/lib/cli_spec.rb

@@ -223,7 +223,7 @@ RSpec.describe Opal::CLI do
     let(:options) { { parse_comments: true, evals: [code], runner: :compiler } }
 
     it 'sets $$comment prop for compiled methods' do
-      expect_output_of { subject.run }.to include('$$comments = ["# multiline", "# comment"]')
+      expect_output_of { subject.run }.to include('$$comments: ["# multiline", "# comment"]')
     end
   end
 

data/spec/lib/compiler_spec.rb

@@ -35,7 +35,7 @@ RSpec.describe Opal::Compiler do
     expect_compiled("123E+10").to include("return 1230000000000")
     expect_compiled("false").to include("return false")
     expect_compiled("true").to include("return true")
-    expect_compiled("nil").to include("return nil")
+    expect_compiled("0;nil").to include("return nil") # NB: Empty nil return is compiled-out
   end
 
   it "should compile ruby strings" do
@@ -66,15 +66,15 @@ RSpec.describe Opal::Compiler do
 
   it "should compile method calls" do
     expect_compiled("self.inspect").to include("$inspect()")
-    expect_compiled("self.map { |a| a + 10 }").to include("$map")
+    expect_compiled("self.map { |a| a + 10 }").to include("'map'")
   end
 
   it "adds method missing stubs" do
-    expect_compiled("self.puts 'hello'").to include("Opal.add_stubs(['$puts'])")
+    expect_compiled("self.puts 'hello'").to include("Opal.add_stubs('puts')")
   end
 
   it 'adds method missing stubs with operators' do
-    expect_compiled("class Foo; end; Foo.new > 5").to include("Opal.add_stubs(['$>', '$new'])")
+    expect_compiled("class Foo; end; Foo.new > 5").to include("Opal.add_stubs('>,new')")
   end
 
   it "should compile constant lookups" do
@@ -99,8 +99,8 @@ RSpec.describe Opal::Compiler do
     end
 
     context "when function name is not valid" do
-      it "skips generating a name" do
-        expect_compiled("def test_method?; end").to include("function()")
+      it "generates a name in a safe way" do
+        expect_compiled("def test_method?; end").to include("function $test_method$ques$1()")
       end
     end
   end
@@ -207,34 +207,42 @@ RSpec.describe Opal::Compiler do
           expect_compiled('foo = 42 if 2 > 3').to include('if ($rb_gt(2, 3))')
           expect_compiled('foo = 42 if 2.5 > 3.5').to include('if ($rb_gt(2.5, 3.5))')
           expect_compiled('foo = 42 if true > false').to include('if ($rb_gt(true, false))')
-
-          expect_compiled('foo = 42 if 2 == 3').to include("if ((2)['$=='](3))")
-          expect_compiled('foo = 42 if 2.5 == 3.5').to include("if ((2.5)['$=='](3.5))")
-          expect_compiled('foo = 42 if true == false').to include("if (true['$=='](false))")
         end
 
         it 'adds nil check for strings' do
           expect_compiled('foo = 42 if "test" > "bar"').to include('if ($truthy($rb_gt("test", "bar")))')
         end
 
-        it 'specifically == excludes nil check for strings' do
-          expect_compiled('foo = 42 if "test" == "bar"').to include("if (\"test\"['$=='](\"bar\"))")
-        end
-
         it 'adds nil check for lvars' do
           expect_compiled("bar = 4\nfoo = 42 if bar > 5").to include('if ($truthy($rb_gt(bar, 5)))')
         end
 
-        it 'specifically == excludes nil check for lvars' do
-          expect_compiled("bar = 4\nfoo = 42 if bar == 5").to include("if (bar['$=='](5))")
+        it 'adds nil check for constants' do
+          expect_compiled("foo = 42 if Test > 4").to include("if ($truthy($rb_gt($$('Test'), 4))) ")
         end
 
-        it 'adds nil check for constants' do
-          expect_compiled("foo = 42 if Test > 4").to include("if ($truthy($rb_gt($$($nesting, 'Test'), 4))) ")
+        it 'converts each == call inside if to an $eqeq wrapper, which does a truthy check' do
+          expect_compiled('foo = 42 if 2 == 3').to include("if ($eqeq(2, 3))")
+          expect_compiled('foo = 42 if 2.5 == 3.5').to include("if ($eqeq(2.5, 3.5))")
+          expect_compiled('foo = 42 if true == false').to include("if ($eqeq(true, false))")
+          expect_compiled('foo = 42 if "test" == "bar"').to include("if ($eqeq(\"test\", \"bar\"))")
+          expect_compiled("bar = 4\nfoo = 42 if bar == 5").to include("if ($eqeq(bar, 5))")
+          expect_compiled("foo = 42 if Test == 4").to include("if ($eqeq($$('Test'), 4))")
+          expect_compiled("bar = 4\nfoo = 42 if bar == 5").to include("if ($eqeq(bar, 5))")
+        end
+
+        it "doesn't convert ==/=== calls to $eqeq(eq) wrappers outside of an if" do
+          expect_compiled("bar == 5").not_to include("$eqeq(bar, 5)")
         end
 
-        it 'specifically == excludes nil check for constants' do
-          expect_compiled("foo = 42 if Test == 4").to include("if ($$($nesting, 'Test')['$=='](4))")
+        it 'converts each === call inside if to an $eqeqeq wrapper, which does a truthy check' do
+          expect_compiled('foo = 42 if 2 === 3').to include("if ($eqeqeq(2, 3))")
+          expect_compiled('foo = 42 if 2.5 === 3.5').to include("if ($eqeqeq(2.5, 3.5))")
+          expect_compiled('foo = 42 if true === false').to include("if ($eqeqeq(true, false))")
+          expect_compiled('foo = 42 if "test" === "bar"').to include("if ($eqeqeq(\"test\", \"bar\"))")
+          expect_compiled("bar = 4\nfoo = 42 if bar === 5").to include("if ($eqeqeq(bar, 5))")
+          expect_compiled("foo = 42 if Test === 4").to include("if ($eqeqeq($$('Test'), 4))")
+          expect_compiled("bar = 4\nfoo = 42 if bar === 5").to include("if ($eqeqeq(bar, 5))")
         end
       end
 
@@ -258,36 +266,28 @@ RSpec.describe Opal::Compiler do
         end
 
         it 'adds nil check for constants' do
-          expect_compiled("foo = 42 if Test").to include("if ($truthy($$($nesting, 'Test')))")
+          expect_compiled("foo = 42 if Test").to include("if ($truthy($$('Test')))")
         end
       end
     end
 
     context 'parentheses' do
       context 'with operators' do
-        it 'adds nil check for primitives' do
-          expect_compiled('foo = 42 if (2 > 3)').to include('if ($truthy($rb_gt(2, 3))')
-          expect_compiled('foo = 42 if (2.5 > 3.5)').to include('if ($truthy($rb_gt(2.5, 3.5))')
-          expect_compiled('foo = 42 if (true > false)').to include('if ($truthy($rb_gt(true, false))')
-
-          expect_compiled('foo = 42 if (2 == 3)').to include("if ($truthy((2)['$=='](3))")
-          expect_compiled('foo = 42 if (2.5 == 3.5)').to include("if ($truthy((2.5)['$=='](3.5))")
-          expect_compiled('foo = 42 if (true == false)').to include("if ($truthy(true['$=='](false)))")
-        end
-
         it 'adds nil check for strings' do
           expect_compiled('foo = 42 if ("test" > "bar")').to include('if ($truthy($rb_gt("test", "bar"))')
-          expect_compiled('foo = 42 if ("test" == "bar")').to include("if ($truthy(\"test\"['$=='](\"bar\"))")
         end
 
         it 'adds nil check for lvars' do
           expect_compiled("bar = 4\nfoo = 42 if (bar > 5)").to include('if ($truthy($rb_gt(bar, 5))')
-          expect_compiled("bar = 4\nfoo = 42 if (bar == 5)").to include("if ($truthy(bar['$=='](5))) ")
         end
 
-        it 'adds nil check for constants' do
-          expect_compiled("foo = 42 if (Test > 4)").to include("if ($truthy($rb_gt($$($nesting, 'Test'), 4))")
-          expect_compiled("foo = 42 if (Test == 4)").to include("if ($truthy($$($nesting, 'Test')['$=='](4))")
+        it 'converts == expressions to $eqeq checks' do
+          expect_compiled('foo = 42 if (2 == 3)').to include("if ($eqeq(2, 3))")
+          expect_compiled('foo = 42 if (2.5 == 3.5)').to include("if ($eqeq(2.5, 3.5))")
+          expect_compiled('foo = 42 if (true == false)').to include("if ($eqeq(true, false))")
+          expect_compiled('foo = 42 if ("test" == "bar")').to include("if ($eqeq(\"test\", \"bar\")")
+          expect_compiled("bar = 4\nfoo = 42 if (bar == 5)").to include("if ($eqeq(bar, 5))")
+          expect_compiled("foo = 42 if (Test == 4)").to include("if ($eqeq($$('Test'), 4))")
         end
       end
 
@@ -311,7 +311,7 @@ RSpec.describe Opal::Compiler do
         end
 
         it 'adds nil check for constants' do
-          expect_compiled("foo = 42 if (Test)").to include("if ($truthy($$($nesting, 'Test'))")
+          expect_compiled("foo = 42 if (Test)").to include("if ($truthy($$('Test'))")
         end
       end
     end

data/spec/lib/simple_server_spec.rb

@@ -47,10 +47,10 @@ RSpec.describe Opal::SimpleServer do
 
   it 'respects config set in Opal::Config' do
     Opal::Config.arity_check_enabled = false
-    expect(get('/assets/console.js').body).not_to include('$Console_clear$1.$$parameters = []')
+    expect(get('/assets/console.js').body).not_to include('$$parameters: []')
 
     Opal::Config.arity_check_enabled = true
     self.app = described_class.new(main: 'console')
-    expect(get('/assets/console.js').body).to include('$Console_clear$1.$$parameters = []')
+    expect(get('/assets/console.js').body).to include('$$parameters: []')
   end
 end

data/spec/lib/source_map/file_spec.rb

@@ -19,7 +19,7 @@ RSpec.describe Opal::SourceMap::File do
       fragment(line: nil, column: nil, source_map_name: nil, code: "\n  ", sexp_type: :top),
       fragment(line: nil, column: nil, source_map_name: nil, code: "var self = Opal.top, $nesting = [], nil = Opal.nil, $$$ = Opal.$$$, $$ = Opal.$$, $breaker = Opal.breaker, $slice = Opal.slice;\n", sexp_type: :top),
       fragment(line: nil, column: nil, source_map_name: nil, code: "\n  ", sexp_type: :top),
-      fragment(line: nil, column: nil, source_map_name: nil, code: "Opal.add_stubs(['$puts']);", sexp_type: :top),
+      fragment(line: nil, column: nil, source_map_name: nil, code: "Opal.add_stubs('puts');", sexp_type: :top),
       fragment(line: nil, column: nil, source_map_name: nil, code: "\n  ", sexp_type: :top),
       fragment(line: 1, column: 0, source_map_name: nil, code: "\n  ", sexp_type: :begin),
       fragment(line: nil, column: nil, source_map_name: "self", code: "self", sexp_type: :self),

data/spec/opal/compiler/irb_spec.rb

@@ -5,14 +5,14 @@ describe Opal::Compiler do
   describe "irb parser option" do
     it "creates Opal.irb_vars if it does not exist" do
       $global["Opal"].irb_vars = nil
-      eval_js compile("nil", :irb => true)
+      eval_js compile("0;nil", :irb => true)
 
       ($global["Opal"].irb_vars == nil).should be_false
     end
 
     it "does not create Opal.irb_vars if :irb option not passed" do
       $global["Opal"].irb_vars = nil
-      eval_js compile("nil")
+      eval_js compile("0;nil")
 
       ($global["Opal"].irb_vars == nil).should be_true
     end

data/spec/opal/core/kernel/puts_spec.rb

@@ -0,0 +1,90 @@
+describe "IO#puts" do
+  before :each do
+    @before_separator = $/
+    @io = IO.new(123)
+    ScratchPad.record []
+    def @io.write(str)
+      ScratchPad << str
+    end
+  end
+
+  after :each do
+    ScratchPad.clear
+    @io.close if @io
+    suppress_warning {$/ = @before_separator}
+  end
+
+  it "writes just a newline when given no args" do
+    @io.puts.should == nil
+    ScratchPad.recorded.join("").should == "\n"
+  end
+
+  it "writes just a newline when given just a newline" do
+    @io.puts("\n").should == nil
+    ScratchPad.recorded.should == ["\n"]
+  end
+
+  it "writes empty string with a newline when given nil as an arg" do
+    @io.puts(nil).should == nil
+    ScratchPad.recorded.join("").should == "\n"
+  end
+
+  it "writes empty string with a newline when when given nil as multiple args" do
+    @io.puts(nil, nil).should == nil
+    ScratchPad.recorded.join("").should == "\n\n"
+  end
+
+  it "calls :to_s before writing non-string objects that don't respond to :to_ary" do
+    object = mock('hola')
+    object.should_receive(:to_s).and_return("hola")
+
+    @io.puts(object).should == nil
+    ScratchPad.recorded.join("").should == "hola\n"
+  end
+
+  # it "returns general object info if :to_s does not return a string" do
+  #   object = mock('hola')
+  #   object.should_receive(:to_s).and_return(false)
+  #
+  #   @io.puts(object).should == nil
+  #   ScratchPad.recorded.join("").should == object.inspect.split(" ")[0] + ">\n"
+  # end
+
+  it "writes each arg if given several" do
+    @io.puts(1, "two", 3).should == nil
+    ScratchPad.recorded.join("").should == "1\ntwo\n3\n"
+  end
+
+  it "flattens a nested array before writing it" do
+    @io.puts([1, 2, [3]]).should == nil
+    ScratchPad.recorded.join("").should == "1\n2\n3\n"
+  end
+
+  it "writes nothing for an empty array" do
+    @io.puts([]).should == nil
+    ScratchPad.recorded.should == []
+  end
+
+  # it "writes [...] for a recursive array arg" do
+  #   x = []
+  #   x << 2 << x
+  #   @io.puts(x).should == nil
+  #   ScratchPad.recorded.join("").should == "2\n[...]\n"
+  # end
+
+  it "writes a newline after objects that do not end in newlines" do
+    @io.puts(5).should == nil
+    ScratchPad.recorded.join("").should == "5\n"
+  end
+
+  it "does not write a newline after objects that end in newlines" do
+    @io.puts("5\n").should == nil
+    ScratchPad.recorded.join("").should == "5\n"
+  end
+
+  it "ignores the $/ separator global" do
+    suppress_warning {$/ = ":"}
+    @io.puts(5).should == nil
+    ScratchPad.recorded.join("").should == "5\n"
+  end
+end

data/spec/opal/core/language/super_spec.rb

@@ -0,0 +1,24 @@
+describe "a block with super" do
+  it "can be used to define multiple methods" do
+    block = proc {
+      super()
+    }
+
+    c1 = Class.new {
+      def foo; :foo; end
+      def bar; :bar; end
+      def foo_bar; [foo, bar]; end
+    }
+
+    c2 = Class.new(c1) {
+      define_method :foo, block
+      define_method :bar, block
+      define_method :foo_bar, block
+    }
+
+    obj = c2.new
+    obj.foo.should == :foo
+    obj.bar.should == :bar
+    obj.foo_bar.should == [:foo, :bar]
+  end
+end

data/spec/ruby_specs

@@ -39,8 +39,8 @@ ruby/core/complex
 
 ruby/core/enumerable
 ruby/core/enumerator
-!ruby/core/enumerator/arithmetic_sequence
-!ruby/core/enumerator/chain
+ruby/core/enumerator/arithmetic_sequence
+ruby/core/enumerator/chain
 ruby/core/enumerator/generator
 ruby/core/enumerator/lazy
 ruby/core/enumerator/yielder
@@ -198,7 +198,9 @@ ruby/library/matrix/vector
 ruby/library/pp
 ruby/library/securerandom
 ruby/library/set
+ruby/library/shellwords
 ruby/library/singleton
 ruby/library/stringio/each_line_spec
 ruby/library/stringio/each_spec
 ruby/library/stringscanner
+ruby/library/optionparser

data/spec/support/rewriters_helper.rb

@@ -29,7 +29,7 @@ module RewritersHelper
   def parse_without_rewriting(source)
     buffer = Opal::Parser::SourceBuffer.new('(eval)')
     buffer.source = source
-    parser = Parser::Ruby30.new
+    parser = Parser::Ruby31.new
     parser.parse(buffer)
   end
 end

data/stdlib/bigdecimal.rb

@@ -67,8 +67,6 @@ class BigDecimal < Numeric
     end
   end
 
-  alias === ==
-
   def <=>(other)
     result = case other
              when self.class
@@ -123,8 +121,6 @@ class BigDecimal < Numeric
     self.class.new(result)
   end
 
-  alias + add
-
   def ceil(n = nil)
     unless bignumber.JS.isFinite
       raise FloatDomainError, "Computation results to 'Infinity'"
@@ -188,8 +184,6 @@ class BigDecimal < Numeric
     self.class.new(bignumber.JS.minus(other.bignumber))
   end
 
-  alias - minus
-
   def mult(other, digits = nil)
     other, _ = coerce(other)
 
@@ -200,8 +194,6 @@ class BigDecimal < Numeric
     self.class.new(bignumber.JS.times(other.bignumber).JS.round(digits, self.class.mode(ROUND_MODE)))
   end
 
-  alias * mult
-
   def nan?
     bignumber.JS.isNaN
   end
@@ -211,8 +203,6 @@ class BigDecimal < Numeric
     self.class.new(bignumber.JS.dividedBy(other.bignumber))
   end
 
-  alias / quo
-
   def sign
     if bignumber.JS.isNaN
       return SIGN_NaN
@@ -234,9 +224,15 @@ class BigDecimal < Numeric
   def to_s(s = '')
     bignumber.JS.toString
   end
-  alias inspect to_s
 
   def zero?
     bignumber.JS.isZero
   end
+
+  alias === ==
+  alias + add
+  alias - minus
+  alias * mult
+  alias / quo
+  alias inspect to_s
 end

data/stdlib/buffer/view.rb

@@ -27,8 +27,6 @@ class Buffer
       `#{@native}.byteLength`
     end
 
-    alias size length
-
     def get(offset, bits = 8, type = :unsigned, little = false)
       `#{@native}["get" + #{Buffer.name_for bits, type}](offset, little)`
     end
@@ -104,5 +102,7 @@ class Buffer
     def set_float64(offset, value, little = false)
       `#{@native}.setFloat64(offset, value, little)`
     end
+
+    alias size length
   end
 end

data/stdlib/buffer.rb

@@ -30,8 +30,6 @@ class Buffer
     `#{@native}.byteLength`
   end
 
-  alias size length
-
   def to_a(bits = 8, type = :unsigned)
     Array.new(self, bits, type)
   end
@@ -43,4 +41,6 @@ class Buffer
   def to_s
     to_a.to_a.pack('c*')
   end
+
+  alias size length
 end

data/stdlib/date.rb

@@ -83,8 +83,6 @@ class Date
   ABBR_DAYNAMES   = %w[Sun Mon Tue Wed Thu Fri Sat]
 
   class << self
-    alias civil new
-
     def wrap(native)
       instance = allocate
       `#{instance}.date = #{native}`
@@ -344,6 +342,8 @@ class Date
     def gregorian_leap?(year)
       `(new Date(#{year}, 1, 29).getMonth()-1) === 0`
     end
+
+    alias civil new
   end
 
   def initialize(year = -4712, month = 1, day = 1, start = ITALY)
@@ -472,8 +472,6 @@ class Date
     }
   end
 
-  alias eql? ==
-
   def clone
     Date.wrap(`new Date(#{@date}.getTime())`)
   end
@@ -588,8 +586,6 @@ class Date
     }
   end
 
-  alias succ next
-
   def sunday?
     wday == 0
   end
@@ -680,4 +676,7 @@ class Date
       return [31, (leap ? 29 : 28), 31, 30, 31, 30, 31, 31, 30, 31, 30, 31][month]
     }
   }
+
+  alias eql? ==
+  alias succ next
 end

data/stdlib/erb.rb

@@ -10,6 +10,7 @@ class ERB
     end
 
     alias h html_escape
+
     module_function :h
     module_function :html_escape
   end

data/stdlib/js.rb

@@ -58,11 +58,12 @@ module JS
     args << block if block
     g.JS[func].JS.apply(g, args)
   end
-  alias method_missing call
 
   def [](name)
     `Opal.global[#{name}]`
   end
 
+  alias method_missing call
+
   extend self
 end

data/stdlib/native.rb

@@ -261,10 +261,6 @@ class Native::Object < BasicObject
     `Opal.hasOwnProperty.call(#{@native}, #{name})`
   end
 
-  alias key? has_key?
-  alias include? has_key?
-  alias member? has_key?
-
   def each(*args)
     if block_given?
       %x{
@@ -341,8 +337,6 @@ class Native::Object < BasicObject
     `Opal.is_a(self, klass)`
   end
 
-  alias kind_of? is_a?
-
   def instance_of?(klass)
     `self.$$class === klass`
   end
@@ -358,6 +352,11 @@ class Native::Object < BasicObject
   def inspect
     "#<Native:#{`String(#{@native})`}>"
   end
+
+  alias include? has_key?
+  alias key? has_key?
+  alias kind_of? is_a?
+  alias member? has_key?
 end
 
 class Native::Array
@@ -435,11 +434,11 @@ class Native::Array
     `#{@native}[#{@length}]`
   end
 
-  alias to_ary to_a
-
   def inspect
     to_a.inspect
   end
+
+  alias to_ary to_a
 end
 
 class Numeric

data/stdlib/nodejs/argf.rb

@@ -99,12 +99,12 @@ class << ARGF
     file.eof?
   end
 
-  alias to_io file
-  alias to_i fileno
-  alias skip close
-  alias path filename
   alias each_line each
   alias eof eof?
+  alias path filename
+  alias skip close
+  alias to_i fileno
+  alias to_io file
 end
 
 ARGF.lineno = 1