From 9b4aa14ebc4ae28861cc76ef80bb671899878ec5 Mon Sep 17 00:00:00 2001 From: Prashant Anantharaman Date: Sun, 11 Sep 2016 09:36:28 -0700 Subject: [PATCH 1/6] Add *.gem to Gemfile --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 570fbf8..8101f08 100644 --- a/.gitignore +++ b/.gitignore @@ -24,3 +24,4 @@ libhammer.pc .sconsign.dblite *.os *.pyc +*.gem From 7b4d208bf31c0ceacc730d477eaae0527cfa4abf Mon Sep 17 00:00:00 2001 From: Prashant Anantharaman Date: Sun, 11 Sep 2016 09:48:05 -0700 Subject: [PATCH 2/6] Change filenames to hammer-parser.rb --- src/bindings/ruby/lib/{hammer.rb => hammer-parser.rb} | 0 src/bindings/ruby/test/autogen_test.rb | 2 +- src/bindings/ruby/test/parser_test.rb | 2 +- 3 files changed, 2 insertions(+), 2 deletions(-) rename src/bindings/ruby/lib/{hammer.rb => hammer-parser.rb} (100%) diff --git a/src/bindings/ruby/lib/hammer.rb b/src/bindings/ruby/lib/hammer-parser.rb similarity index 100% rename from src/bindings/ruby/lib/hammer.rb rename to src/bindings/ruby/lib/hammer-parser.rb diff --git a/src/bindings/ruby/test/autogen_test.rb b/src/bindings/ruby/test/autogen_test.rb index 0600c0f..2efc6a6 100644 --- a/src/bindings/ruby/test/autogen_test.rb +++ b/src/bindings/ruby/test/autogen_test.rb @@ -1,6 +1,6 @@ require 'bundler/setup' require 'minitest/autorun' -require 'hammer' +require 'hammer-parser' class TestToken < Minitest::Test def setup super diff --git a/src/bindings/ruby/test/parser_test.rb b/src/bindings/ruby/test/parser_test.rb index 6bbdc36..78938fc 100644 --- a/src/bindings/ruby/test/parser_test.rb +++ b/src/bindings/ruby/test/parser_test.rb @@ -1,6 +1,6 @@ # -*- coding: utf-8 -*- require 'bundler/setup' -require 'hammer' +require 'hammer-parser' require 'minitest/autorun' class ParserTest < Minitest::Test From 971e4e4f70ff87da7609845f27a349a9ec5e00d4 Mon Sep 17 00:00:00 2001 From: Prashant Anantharaman Date: Sun, 11 Sep 2016 10:05:18 -0700 Subject: [PATCH 3/6] Change documentation and examples to reflect filename change --- src/bindings/ruby/README.md | 2 +- src/bindings/ruby/lib/hammer-parser.rb | 5 ----- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/src/bindings/ruby/README.md b/src/bindings/ruby/README.md index 480f255..849db21 100644 --- a/src/bindings/ruby/README.md +++ b/src/bindings/ruby/README.md @@ -50,7 +50,7 @@ Add hammer to your Gemfile. Use hammer in your project. - `require 'hammer'` + `require 'hammer-parser'` ### Building a parser diff --git a/src/bindings/ruby/lib/hammer-parser.rb b/src/bindings/ruby/lib/hammer-parser.rb index 916a0a5..6622337 100644 --- a/src/bindings/ruby/lib/hammer-parser.rb +++ b/src/bindings/ruby/lib/hammer-parser.rb @@ -2,11 +2,6 @@ require 'hammer/internal' require 'hammer/parser' require 'hammer/parser_builder' -# TODO: -# Probably need to rename this file to 'hammer-parser.rb', so -# people can use "require 'hammer-parser'" in their code. - - # Leave this in for now to be able to play around with HParseResult in irb. x = nil parser = Hammer::Parser.build { From 262675fa4e47e54ae0dec07f0f93723c88c78c8c Mon Sep 17 00:00:00 2001 From: Prashant Date: Mon, 24 Oct 2016 23:19:51 -0400 Subject: [PATCH 4/6] Change version number to reflect change of filenames --- src/bindings/ruby/hammer-parser.gemspec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bindings/ruby/hammer-parser.gemspec b/src/bindings/ruby/hammer-parser.gemspec index 9a4a616..58f5484 100644 --- a/src/bindings/ruby/hammer-parser.gemspec +++ b/src/bindings/ruby/hammer-parser.gemspec @@ -1,7 +1,7 @@ #encoding: UTF-8 Gem::Specification.new do |s| s.name = 'hammer-parser' - s.version = '0.1.0' + s.version = '0.2.0' s.summary = 'Ruby bindings to the hammer parsing library.' s.description = s.summary # TODO: longer description? s.authors = ['Meredith L. Patterson', 'TQ Hirsch', 'Jakob Rath'] From f1810153c5a2e88d1adeed4661da3bd669f3f151 Mon Sep 17 00:00:00 2001 From: Prashant Date: Mon, 24 Oct 2016 23:20:47 -0400 Subject: [PATCH 5/6] Fix the filenames in the gemspec file --- src/bindings/ruby/hammer-parser.gemspec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bindings/ruby/hammer-parser.gemspec b/src/bindings/ruby/hammer-parser.gemspec index 58f5484..158bc26 100644 --- a/src/bindings/ruby/hammer-parser.gemspec +++ b/src/bindings/ruby/hammer-parser.gemspec @@ -15,7 +15,7 @@ Gem::Specification.new do |s| "lib/hammer/internal.rb", "lib/hammer/parser.rb", "lib/hammer/parser_builder.rb", - "lib/hammer.rb", + "lib/hammer-parser.rb", "lib/minitest/hamer-parser_plugin.rb", "test/autogen_test.rb", "test/parser_test.rb" From 6dd481de1f9a9980955cc54741581fd897c3008c Mon Sep 17 00:00:00 2001 From: Prashant Date: Sat, 5 Nov 2016 20:37:08 -0400 Subject: [PATCH 6/6] Add attr_reader for dont_gc --- src/bindings/ruby/lib/hammer/parser.rb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/bindings/ruby/lib/hammer/parser.rb b/src/bindings/ruby/lib/hammer/parser.rb index d1177c5..2f19e15 100644 --- a/src/bindings/ruby/lib/hammer/parser.rb +++ b/src/bindings/ruby/lib/hammer/parser.rb @@ -19,8 +19,10 @@ module Hammer @dont_gc = dont_gc.dup end + # dont_gc is required to build a fuzzer from the declaration of Hammer::Parser object. attr_reader :name attr_reader :h_parser + attr_reader :dont_gc # Parse the given data. Returns the parse result if successful, nil otherwise. #