opal 0.8.0.beta1 → 0.8.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4a0a405fd05a0fa6b1f6efac707d0317c6d0ca57
-  data.tar.gz: c57f69d7c54f38d210cc3bfb139c98af5385505b
+  metadata.gz: 3afa0a5d5b918de046d4985b97fa8464b5e73014
+  data.tar.gz: a0a71fde911085d35f81cf363dd1d6569cb2119f
 SHA512:
-  metadata.gz: a2e4b8e88e96a05e3f5a284624571b6e8232594782cc72189ed48835d30ab3cd32ca0ab9d40e3cc93e2978fd1a1c0c4750408f8a57bf55a8aba07eee80278d50
-  data.tar.gz: 902b292438b1fe4893f45a52c3dc31429aa9eebefda5914cd18705c863d6e308ab36ea1323320ba2b9374f6360037a5a55dd9425488b7155b5aab97517a59324
+  metadata.gz: a64a3b104c5b74e7fece49e701449e0bc69d55883848622e5697cd69aa228203ad78ca8d692af4da0d81bc3b950b31cfbbb053c99a2ccfa0fe7048aded6787c1
+  data.tar.gz: c1f7ca3700b01bac88d7c401d7d2affe59985738415a9dd82aebf2217943d81d61a50e37827f1114a727bf30b33e4a511ffad3ab7e8b606cc1c466023c10ddd8

data/.gitmodules

@@ -1,30 +1,6 @@
 [submodule "spec/corelib"]
 	path = spec/corelib
 	url = https://github.com/opal/rubyspec
-[submodule "spec/stdlib/rubysl-singleton"]
-	path = spec/stdlib/rubysl-singleton
-	url = https://github.com/rubysl/rubysl-singleton
-[submodule "spec/stdlib/rubysl-observer"]
-	path = spec/stdlib/rubysl-observer
-	url = https://github.com/rubysl/rubysl-observer
-[submodule "spec/stdlib/rubysl-strscan"]
-	path = spec/stdlib/rubysl-strscan
-	url = https://github.com/rubysl/rubysl-strscan
-[submodule "spec/stdlib/rubysl-delegate"]
-	path = spec/stdlib/rubysl-delegate
-	url = https://github.com/rubysl/rubysl-delegate
-[submodule "spec/stdlib/rubysl-erb"]
-	path = spec/stdlib/rubysl-erb
-	url = https://github.com/rubysl/rubysl-erb
-[submodule "spec/stdlib/rubysl-set"]
-	path = spec/stdlib/rubysl-set
-	url = https://github.com/rubysl/rubysl-set
-[submodule "spec/stdlib/rubysl-pathname"]
-	path = spec/stdlib/rubysl-pathname
-	url = https://github.com/rubysl/rubysl-pathname
-[submodule "spec/stdlib/rubysl-date"]
-	path = spec/stdlib/rubysl-date
-	url = https://github.com/rubysl/rubysl-date
 [submodule "test/cruby"]
 	path = test/cruby
 	url = https://github.com/ruby/ruby.git

data/.jshintrc

@@ -0,0 +1,51 @@
+{
+  // http://jshint.com/docs/options
+
+  "curly"         : true, // Require curly braces around blocks in loops and conditionals
+  "eqeqeq"        : true, // Prohibit the use of == and != in favor of === and !==
+  "es3"           : true, // Adhere to ECMAScript 3 specification
+  "forin"         : true, // Require all for in loops to filter object's items (hasOwnProperty)
+  "latedef"       : true, // Prohibit the use of a variable before it was defined
+  "noarg"         : true, // Prohibit the use of arguments.caller and arguments.callee
+  "nonbsp"        : true, // Warn about "non-breaking whitespace" characters
+  "undef"         : true, // Prohibit the use of explicitly undeclared variables
+  "unused"        : true, // Warn when you define and never use your variables
+
+  "globals": {
+    "Opal": true,
+    "OpalNode": true,
+    "callPhantom": true,
+    "JSON": true
+  },
+
+  "browser": true,
+  "node": true,
+
+  // Currently the following checks are failing:
+
+  "-W003": false, // 'variable' was used before it was defined
+  "-W004": false, // 'variable' is already defined
+  "-W018": false, // Confusing use of '!'
+  "-W021": false, // '$ClassName' is a function
+  "-W024": false, // Expected an identifier and instead saw 'eval' (a reserved word)
+  "-W027": false, // Unreachable ';' after 'return'
+  "-W032": false, // Unnecessary semicolon
+  "-W030": false, // Expected an assignment or function call and instead saw an expression
+  "-W033": false, // Missing semicolon
+  "-W038": false, // 'variable' used out of scope
+  "-W041": false, // Use '===' to compare with 'true'
+  "-W049": false, // Unexpected escaped character '<' in regular expression
+  "-W053": false, // Do not use String as a constructor
+  "-W058": false, // Missing '()' invoking a constructor
+  "-W061": false, // eval can be harmful
+  "-W065": false, // Missing radix parameter
+  "-W069": false, // ['prop'] is better written in dot notation
+  "-W080": false, // It's not necessary to initialize 'result' to 'undefined'
+  "-W083": false, // Don't make functions within a loop
+  "-W086": false, // Expected a 'break' statement before 'case'
+  "-W089": false, // The body of a for in should be wrapped in an if statement to filter unwanted properties from the prototype
+  "-W093": false, // Did you mean to return a conditional instead of an assignment?
+  "-W098": false, // 'variable' is defined but never used
+  "-W116": false, // Expected '===' and instead saw '=='
+  "-W120": false  // You might be leaking a variable ($ClassName) here
+}

data/.travis.yml

@@ -13,7 +13,7 @@ matrix:
       env: RUN=default TZ="/usr/share/zoneinfo/Pacific/Fiji"
 
     - rvm: 2.2
-      env: RUN=default SPROCKETS_VERSION="~> 2.12.3"
+      env: RUN=rspec TILT_VERSION=2.0.1
 
     - rvm: 2.1
       env: RUN=rspec
@@ -33,6 +33,11 @@ matrix:
     - rvm: jruby-head
       env: RUN=rspec
 
+    - rvm: 2.2
+      script:
+        - npm install -g jshint
+        - bundle exec rake lint
+
   allow_failures:
     - rvm: 1.8.7
     - rvm: 1.9.3

data/CHANGELOG.md

@@ -1,5 +1,7 @@
 ## edge (upcoming 0.8.0)
 
+*   Removed minitest from stdlib. It's not part of MRI and it never belonged there, checkout the opal-minitest gem instead.
+
 *   Parser: Allow trailing comma in paren arglists, after normal args as
     well as assoc args.
 

data/Gemfile

@@ -9,15 +9,14 @@ gem 'json', '< 1.8.1'  if RUBY_VERSION.to_f == 2.1 and RUBY_ENGINE == 'ruby'
 gem 'rubysl', :platform => :rbx
 gem 'thin', platform: :mri
 
-# Uncomment to try with sprockets 3.0:
-#
-#   gem 'sprockets', '~> 3.0.0.beta'
-
 group :repl do
   gem 'therubyracer', :platform => :mri, :require => 'v8'
   gem 'therubyrhino', :platform => :jruby
 end
 
+tilt_version = ENV['TILT_VERSION']
+gem 'tilt', tilt_version if tilt_version
+
 unless ENV['CI']
   gem 'rb-fsevent'
   gem 'guard', require: false
@@ -25,6 +24,3 @@ unless ENV['CI']
 end
 
 gem 'mspec', github: 'rubyspec/mspec'
-
-sprockets_version = ENV['SPROCKETS_VERSION']
-gem 'sprockets', sprockets_version if sprockets_version

data/README.md

@@ -7,9 +7,8 @@
 Opal is a ruby to javascript source-to-source compiler. It also has an
 implementation of the ruby corelib.
 
-Opal is [hosted on github](http://github.com/opal/opal), and there
-is a Freenode IRC channel at [#opal](http://webchat.freenode.net/?channels=opal),
-ask questions on [stackoverflow (tag #opalrb)](http://stackoverflow.com/questions/ask?tags=opalrb).
+Opal is [hosted on github](http://github.com/opal/opal). Chat is available on *Gitter* at [opal/opal](https://gitter.im/opal/opal) and the Freenode IRC channel at [#opal](http://webchat.freenode.net/?channels=opal).
+Ask questions on [stackoverflow (tag #opalrb)](http://stackoverflow.com/questions/ask?tags=opalrb). Get the [Opalist newsletter](http://opalist.co) for updates and community news.
 
 [![Inline docs](http://inch-ci.org/github/opal/opal.svg?branch=master&style=flat)](http://opalrb.org/docs/api)
 [![Gitter chat](http://img.shields.io/badge/gitter-opal%2Fopal-009966.svg?style=flat)](https://gitter.im/opal/opal)

data/Rakefile

@@ -9,5 +9,6 @@ import 'tasks/github.rake'
 import 'tasks/documenting.rake'
 import 'tasks/testing.rake'
 import 'tasks/building.rake'
+import 'tasks/linting.rake'
 
 task :default => [:rspec, :mspec_node, :cruby_tests]

data/lib/mspec/opal/rake_task.rb

@@ -163,7 +163,7 @@ module MSpec
 
       def paths_from_glob pattern
         Dir.glob(File.expand_path(pattern)).map do |s|
-          s.sub(/^#{basedir}\//, '').sub(/\.rb$/, '')
+          s.sub(/\A#{basedir}\//, '').sub(/\.rb\z/, '')
         end
       end
 

data/lib/opal/builder.rb

@@ -1,5 +1,6 @@
 require 'opal/path_reader'
 require 'opal/builder_processors'
+require 'opal/paths'
 require 'set'
 
 module Opal
@@ -42,13 +43,24 @@ module Opal
       processed << asset
       self
     rescue MissingRequire => error
-      raise error, "A file required by #{filename.inspect} wasn't found.\n#{error.message}"
+      raise error, "A file required by #{filename.inspect} wasn't found.\n#{error.message}", error.backtrace
     end
 
     def build_require(path, options = {})
       process_require(path, options)
     end
 
+    def initialize_copy(other)
+      super
+      @stubs = other.stubs.dup
+      @preload = other.preload.dup
+      @processors = other.processors.dup
+      @path_reader = other.path_reader.dup
+      @prerequired = other.prerequired.dup
+      @compiler_options = other.compiler_options.dup
+      @processed = other.processed.dup
+    end
+
     def to_s
       processed.map(&:to_s).join("\n")
     end
@@ -57,6 +69,12 @@ module Opal
       processed.map(&:source_map).reduce(:+).as_json.to_json
     end
 
+    def append_paths(*paths)
+      path_reader.append_paths(*paths)
+    end
+
+    include UseGem
+
     attr_reader :processed
 
     attr_accessor :processors, :default_processor, :path_reader,
@@ -84,7 +102,7 @@ module Opal
         globs = extensions.map { |ext| File.join base, tree, '**', "*.#{ext}" }
 
         Dir[*globs].map do |file|
-          Pathname(file).relative_path_from(Pathname(base)).to_s.gsub(/(\.js)?(\.(?:#{extensions.join '|'}))$/, '')
+          Pathname(file).relative_path_from(Pathname(base)).to_s.gsub(/(\.js)?(\.(?:#{extensions.join '|'}))#{REGEXP_END}/, '')
         end
       end
     end
@@ -101,7 +119,7 @@ module Opal
     end
 
     def process_require(filename, options)
-      filename.gsub!(/\.(rb|js|opal)$/, '')
+      filename.gsub!(/\.(rb|js|opal)#{REGEXP_END}/, '')
       return if prerequired.include?(filename)
       return if already_processed.include?(filename)
       already_processed << filename

data/lib/opal/builder_processors.rb

@@ -27,7 +27,7 @@ module Opal
         matches = extensions.join('|')
         matches = "(#{matches})" if extensions.size == 1
 
-        @match_regexp = Regexp.new "\\.#{matches}$"
+        @match_regexp = Regexp.new "\\.#{matches}#{REGEXP_END}"
       end
 
       def self.extensions
@@ -87,7 +87,7 @@ module Opal
 
       def compiled
         @compiled ||= begin
-          compiler = compiler_for(@source, file: @filename.gsub(/\.(rb|js|opal)$/, ''))
+          compiler = compiler_for(@source, file: @filename.gsub(/\.(rb|js|opal)#{REGEXP_END}/, ''))
           compiler.compile
           compiler
         end

data/lib/opal/cli.rb

@@ -87,11 +87,10 @@ module Opal
 
     attr_reader :exit_status
 
-    def compiled_source
-      Opal.paths.concat load_paths
-      gems.each { |gem_name| Opal.use_gem gem_name }
-
+    def build
       builder = Opal::Builder.new stubs: stubs, compiler_options: compiler_options
+      builder.append_paths(*load_paths)
+      gems.each { |gem_name| builder.use_gem gem_name }
 
       builder.build 'opal' unless skip_opal_require?
 
@@ -116,7 +115,11 @@ module Opal
 
       builder.build_str 'Kernel.exit', '(exit)' unless no_exit
 
-      builder.to_s
+      builder
+    end
+
+    def compiled_source
+      build.to_s
     end
 
     def show_compiled_source

data/lib/opal/erb.rb

@@ -76,7 +76,7 @@ module Opal
       end
 
       def wrap_compiled(result)
-        path = @file_name.sub(/\.opalerb$/, '')
+        path = @file_name.sub(/\.opalerb#{REGEXP_END}/, '')
         result = "Template.new('#{path}') do |output_buffer|\noutput_buffer.append(\"#{result}\")\noutput_buffer.join\nend\n"
       end
     end

data/lib/opal/nodes/call.rb

@@ -90,7 +90,7 @@ module Opal
       end
 
       def attr_assignment?
-        @assignment ||= meth.to_s =~ /^[\da-z]+\=$/i
+        @assignment ||= meth.to_s =~ /#{REGEXP_START}[\da-z]+\=#{REGEXP_END}/i
       end
 
       # Used to generate the code to use this sexp as an ivar var reference

data/lib/opal/nodes/call_special.rb

@@ -12,7 +12,7 @@ module Opal
 
       def default_compile
         # Skip, for now, if the method has square brackets: []=
-        return super if meth.to_s !~ /^\w+=$/
+        return super if meth.to_s !~ /#{REGEXP_START}\w+=#{REGEXP_END}/
 
         with_temp do |args_tmp|
           with_temp do |recv_tmp|

data/lib/opal/nodes/helpers.rb

@@ -1,19 +1,21 @@
+require 'opal/regexp_anchors'
+
 module Opal
   module Nodes
     module Helpers
 
       # Reserved javascript keywords - we cannot create variables with the
       # same name (ref: http://stackoverflow.com/a/9337272/601782)
-      ES51_RESERVED_WORD = /^(?:do|if|in|for|let|new|try|var|case|else|enum|eval|false|null|this|true|void|with|break|catch|class|const|super|throw|while|yield|delete|export|import|public|return|static|switch|typeof|default|extends|finally|package|private|continue|debugger|function|arguments|interface|protected|implements|instanceof)$/
+      ES51_RESERVED_WORD = /#{REGEXP_START}(?:do|if|in|for|let|new|try|var|case|else|enum|eval|false|null|this|true|void|with|break|catch|class|const|super|throw|while|yield|delete|export|import|public|return|static|switch|typeof|default|extends|finally|package|private|continue|debugger|function|arguments|interface|protected|implements|instanceof)#{REGEXP_END}/
 
       # ES3 reserved words that aren’t ES5.1 reserved words
-      ES3_RESERVED_WORD_EXCLUSIVE = /^(?:int|byte|char|goto|long|final|float|short|double|native|throws|boolean|abstract|volatile|transient|synchronized)$/
+      ES3_RESERVED_WORD_EXCLUSIVE = /#{REGEXP_START}(?:int|byte|char|goto|long|final|float|short|double|native|throws|boolean|abstract|volatile|transient|synchronized)#{REGEXP_END}/
 
       # Immutable properties of the global object
-      IMMUTABLE_PROPS = /^(?:NaN|Infinity|undefined)$/
+      IMMUTABLE_PROPS = /#{REGEXP_START}(?:NaN|Infinity|undefined)#{REGEXP_END}/
 
       # Doesn't take in account utf8
-      BASIC_IDENTIFIER_RULES = /^[$_a-z][$_a-z\d]*$/i
+      BASIC_IDENTIFIER_RULES = /#{REGEXP_START}[$_a-z][$_a-z\d]*#{REGEXP_END}/i
 
 
       def property(name)

data/lib/opal/nodes/iter.rb

@@ -67,7 +67,7 @@ module Opal
           elsif arg.type == :array
             arg[1..-1].each_with_index do |_arg, _idx|
               _arg = variable(_arg[1])
-              push "#{_arg} = #{params[idx]}[#{_idx}];"
+              push "var #{_arg} = #{params[idx]}[#{_idx}];"
             end
           else
             raise "Bad block arg type"
@@ -115,7 +115,7 @@ module Opal
         sexp.each do |arg|
           if arg[0] == :lasgn
             ref = variable(arg[1])
-            scope.add_arg ref
+            self.add_arg ref
             result << ref
           elsif arg[0] == :array
             result << scope.next_temp

data/lib/opal/nodes/singleton_class.rb

@@ -14,8 +14,9 @@ module Opal
           add_temp '$scope = self.$$scope'
           add_temp 'def = self.$$proto'
 
+          body_stmt = stmt(compiler.returns(body))
           line scope.to_vars
-          line stmt(compiler.returns(body))
+          line body_stmt
         end
 
         line "})(", recv(object), ".$singleton_class())"

data/lib/opal/parser/lexer.rb

@@ -1,3 +1,4 @@
+require 'opal/regexp_anchors'
 require 'strscan'
 require 'opal/parser/keywords'
 
@@ -558,7 +559,7 @@ module Opal
             matched += scanner.matched
           end
 
-        elsif matched =~ /^[A-Z]/
+        elsif matched =~ /#{REGEXP_START}[A-Z]/
           result = :tCONSTANT
         else
           result = :tIDENTIFIER
@@ -627,7 +628,7 @@ module Opal
         @lex_state = :expr_end
       end
 
-      return matched =~ /^[A-Z]/ ? :tCONSTANT : :tIDENTIFIER
+      return matched =~ /#{REGEXP_START}[A-Z]/ ? :tCONSTANT : :tIDENTIFIER
     end
 
     # Does the heavy lifting for `next_token`.

data/lib/opal/path_reader.rb

@@ -13,13 +13,20 @@ module Opal
     end
 
     def expand(path)
-      file_finder.find(path)
+      if Pathname.new(path).absolute? || path =~ %r{\A\.?\.#{File::SEPARATOR}}
+        path
+      else
+        file_finder.find(path)
+      end
     end
 
     def paths
       file_finder.paths
     end
 
+    def append_paths(*paths)
+      file_finder.append_paths(*paths)
+    end
 
     private
 

data/lib/opal/paths.rb

@@ -16,31 +16,38 @@ module Opal
   # here should only be paths which contain code targeted at being compiled by
   # Opal.
   def self.append_path(path)
-    paths << path
+    append_paths(path)
+  end
+  def self.append_paths(*paths)
+    self.paths.concat(paths)
   end
 
-  def self.use_gem(gem_name, include_dependencies = true)
-    require_paths_for_gem(gem_name, include_dependencies).each do |path|
-      append_path path
+  module UseGem
+    def use_gem(gem_name, include_dependencies = true)
+      append_paths(*require_paths_for_gem(gem_name, include_dependencies))
     end
-  end
+    
+    private
 
-  def self.require_paths_for_gem(gem_name, include_dependencies)
-    paths = []
-    spec = Gem::Specification.find_by_name(gem_name)
+    def require_paths_for_gem(gem_name, include_dependencies)
+      paths = []
+      spec = Gem::Specification.find_by_name(gem_name)
 
-    spec.runtime_dependencies.each do |dependency|
-      paths += require_paths_for_gem(dependency.name, include_dependencies)
-    end if include_dependencies
+      spec.runtime_dependencies.each do |dependency|
+        paths += require_paths_for_gem(dependency.name, include_dependencies)
+      end if include_dependencies
 
-    gem_dir = spec.gem_dir
-    spec.require_paths.map do |path|
-      paths << File.join(gem_dir, path)
-    end
+      gem_dir = spec.gem_dir
+      spec.require_paths.map do |path|
+        paths << File.join(gem_dir, path)
+      end
 
-    paths
+      paths
+    end
   end
 
+  extend UseGem
+
   # Private, don't add to these directly (use .append_path instead).
   def self.paths
     @paths ||= [core_dir.untaint, std_dir.untaint, gem_dir.untaint]