From 049a64946b5240fa75bb19a8846427a27d264bb0 Mon Sep 17 00:00:00 2001
From: Jakob Rath <git@jakobrath.eu>
Date: Fri, 15 Nov 2013 14:50:31 +0100
Subject: [PATCH] Fix indentation.
---
src/bindings/ruby/lib/hammer/parser.rb | 50 +++++++++++++-------------
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/src/bindings/ruby/lib/hammer/parser.rb b/src/bindings/ruby/lib/hammer/parser.rb
index defb776b..b5d95c87 100644
--- a/src/bindings/ruby/lib/hammer/parser.rb
+++ b/src/bindings/ruby/lib/hammer/parser.rb
@@ -28,40 +28,40 @@ module Hammer
end
def self.ch(char)
- # TODO: Really? Should probably accept Fixnum in appropriate range
- # Also, char.ord gives unexpected results if you pass e.g. Japanese characters: '今'.ord == 20170; Hammer::Parser::Ch.new('今').parse(202.chr) == true
- # Not really unexpected though, since 20170 & 255 == 202.
- # But probably it's better to use Ch for Fixnum in 0..255 only, and only Token for strings.
- raise ArgumentError, 'expecting a one-character String' unless char.is_a?(String) && char.length == 1
- h_parser = Hammer::Internal.h_ch(char.ord)
+ # TODO: Really? Should probably accept Fixnum in appropriate range
+ # Also, char.ord gives unexpected results if you pass e.g. Japanese characters: '今'.ord == 20170; Hammer::Parser::Ch.new('今').parse(202.chr) == true
+ # Not really unexpected though, since 20170 & 255 == 202.
+ # But probably it's better to use Ch for Fixnum in 0..255 only, and only Token for strings.
+ raise ArgumentError, 'expecting a one-character String' unless char.is_a?(String) && char.length == 1
+ h_parser = Hammer::Internal.h_ch(char.ord)
- parser = Hammer::Parser.new
- parser.instance_variable_set :@h_parser, h_parser
- return parser
+ parser = Hammer::Parser.new
+ parser.instance_variable_set :@h_parser, h_parser
+ return parser
end
def self.sequence(*parsers)
- args = parsers.flat_map { |p| [:pointer, p.h_parser] }
- h_parser = Hammer::Internal.h_sequence(*args, :pointer, nil)
- sub_parsers = parsers # store them so they don't get garbage-collected (probably not needed, though)
- # TODO: Use (managed?) FFI struct instead of void pointers
+ args = parsers.flat_map { |p| [:pointer, p.h_parser] }
+ h_parser = Hammer::Internal.h_sequence(*args, :pointer, nil)
+ sub_parsers = parsers # store them so they don't get garbage-collected (probably not needed, though)
+ # TODO: Use (managed?) FFI struct instead of void pointers
- parser = Hammer::Parser.new
- parser.instance_variable_set :@h_parser, h_parser
- parser.instance_variable_set :@sub_parsers, sub_parsers
- return parser
+ parser = Hammer::Parser.new
+ parser.instance_variable_set :@h_parser, h_parser
+ parser.instance_variable_set :@sub_parsers, sub_parsers
+ return parser
end
def self.choice(*parsers)
- args = parsers.flat_map { |p| [:pointer, p.h_parser] }
- h_parser = Hammer::Internal.h_choice(*args, :pointer, nil)
- sub_parsers = parsers # store them so they don't get garbage-collected (probably not needed, though)
- # TODO: Use (managed?) FFI struct instead of void pointers
+ args = parsers.flat_map { |p| [:pointer, p.h_parser] }
+ h_parser = Hammer::Internal.h_choice(*args, :pointer, nil)
+ sub_parsers = parsers # store them so they don't get garbage-collected (probably not needed, though)
+ # TODO: Use (managed?) FFI struct instead of void pointers
- parser = Hammer::Parser.new
- parser.instance_variable_set :@h_parser, h_parser
- parser.instance_variable_set :@sub_parsers, sub_parsers
- return parser
+ parser = Hammer::Parser.new
+ parser.instance_variable_set :@h_parser, h_parser
+ parser.instance_variable_set :@sub_parsers, sub_parsers
+ return parser
end
# Defines a parser constructor with the given name.
--
GitLab