SgfParser 0.8.0 → 0.9.0

data/.document

@@ -1,5 +1,5 @@
-README.rdoc
-lib/**/*.rb
-bin/*
-features/**/*.feature
-LICENSE
+README.rdoc
+lib/**/*.rb
+bin/*
+features/**/*.feature
+LICENSE

data/.gitignore

@@ -1,25 +1,25 @@
-## MAC OS
-.DS_Store
-
-## TEXTMATE
-*.tmproj
-tmtags
-
-## EMACS
-*~
-\#*
-.\#*
-
-## VIM
-*.swp
-
-## PROJECT::GENERAL
-coverage
-rdoc
-pkg
-
-## PROJECT::SPECIFIC
-
-
-## Rubymine
+## MAC OS
+.DS_Store
+
+## TEXTMATE
+*.tmproj
+tmtags
+
+## EMACS
+*~
+\#*
+.\#*
+
+## VIM
+*.swp
+
+## PROJECT::GENERAL
+coverage
+rdoc
+pkg
+
+## PROJECT::SPECIFIC
+
+
+## Rubymine
 .idea/*

data/README.rdoc

@@ -1,24 +1,34 @@
+=INFORMATION
+Author: Aldric Giacomoni
+
+Email : aldric~at~trevoke.net (feedback very welcome!)
+
 SGF: all formats (but untested with FF < 4)
-Ruby: >1.8.7 (may work with 1.8.6)
 
+Ruby: >=1.8.7 (may work with 1.8.6)
+
+=QUICK HOWTO
 Example:
-require 'sgf_parser'
-tree = SgfParser::Tree.new :filename => File
-tree = SgfParser::Tree.new :sgf_string => String
+ require 'sgf_parser'
+ tree = SgfParser::Tree.new :filename => File
+ tree = SgfParser::Tree.new :string => String
 
 All trees begin with an empty node ( @root) which allows a simple support of multiple gametrees.
+
 Most games will just care about, say,
-tree.root.children[0] which is the first node of the first gametree.
+ tree.root.children[0] which is the first node of the first gametree.
 
 For any node, one can summon the properties as such:
-node.properties # => returns a hash of the properties.
+ node.properties # => returns a hash of the properties.
 A single property can be called, like the comments, for instance, like so:
-node.C # => returns the comments for this node.
+ node.C # => returns the comments for this node.
 
 The library currently uses method_missing to painlessly return the data. I must admit that this is both clever coding and laziness on my part.
 
-The 'SGF Indenter', the purpose of which is to make the actual SGF file more
-human readable, is working.
+There is also a SGF Indenter. Its purpose is to make SGF files more readable to humans at a glance.
+ require 'sgf_parser/sgfindent' # Done automatically if you require 'sgf_parser'
+ sgf = SgfParser::Indenter.new 'some_ugly_file.sgf' # Will output to the console
+ sgf = SgfParser::Indenter.new 'some_ugly_file.sgf' 'pretty.sgf' # Sends the result to a new file.
 
 ___
 

data/SgfParser.gemspec

@@ -1,66 +1,67 @@
-# Generated by jeweler
-# DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run the gemspec command
-# -*- encoding: utf-8 -*-
-
-Gem::Specification.new do |s|
-  s.name = %q{SgfParser}
-  s.version = "0.8.0"
-
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Aldric Giacomoni"]
-  s.date = %q{2010-01-05}
-  s.description = %q{SGFParser is a library that parses and saves SGF (Smart Game Format) files.}
-  s.email = %q{aldric@trevoke.net}
-  s.extra_rdoc_files = [
-    "LICENSE",
-     "README.rdoc"
-  ]
-  s.files = [
-    ".document",
-     ".gitignore",
-     "LICENSE",
-     "README.rdoc",
-     "Rakefile",
-     "SgfParser.gemspec",
-     "VERSION",
-     "lib/sgf/parser/node.rb",
-     "lib/sgf/parser/properties.rb",
-     "lib/sgf/parser/tree.rb",
-     "lib/sgf/parser/tree_parse.rb",
-     "lib/sgf/sgfindent.rb",
-     "lib/sgf_parser.rb",
-     "sample_sgf/ff4_ex.sgf",
-     "sample_sgf/ff4_ex_saved.sgf",
-     "sample_sgf/redrose-tartrate.sgf",
-     "sample_usage/parsing_files.rb",
-     "spec/node_spec.rb",
-     "spec/spec.opts",
-     "spec/spec_helper.rb",
-     "spec/tree_spec.rb"
-  ]
-  s.homepage = %q{http://github.com/Trevoke/SGFParser}
-  s.rdoc_options = ["--charset=UTF-8"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.3.5}
-  s.summary = %q{A library for working with SGF files.}
-  s.test_files = [
-    "spec/node_spec.rb",
-     "spec/spec_helper.rb",
-     "spec/tree_spec.rb"
-  ]
-
-  if s.respond_to? :specification_version then
-    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
-    s.specification_version = 3
-
-    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
-    else
-      s.add_dependency(%q<rspec>, [">= 1.2.9"])
-    end
-  else
-    s.add_dependency(%q<rspec>, [">= 1.2.9"])
-  end
-end
-
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in rakefile, and run the gemspec command
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = %q{SgfParser}
+  s.version = "0.9.0"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["Aldric Giacomoni"]
+  s.date = %q{2010-04-30}
+  s.description = %q{SGFParser is a library that parses and saves SGF (Smart Game Format) files.}
+  s.email = %q{aldric@trevoke.net}
+  s.extra_rdoc_files = [
+    "LICENSE",
+     "README.rdoc"
+  ]
+  s.files = [
+    ".document",
+     ".gitignore",
+     "LICENSE",
+     "README.rdoc",
+     "Rakefile",
+     "SgfParser.gemspec",
+     "VERSION",
+     "lib/sgf/parser/node.rb",
+     "lib/sgf/parser/properties.rb",
+     "lib/sgf/parser/tree.rb",
+     "lib/sgf/parser/tree_parse.rb",
+     "lib/sgf/sgf_indent.rb",
+     "lib/sgf_parser.rb",
+     "sample_sgf/ff4_ex.sgf",
+     "sample_sgf/redrose-tartrate.sgf",
+     "sample_usage/parsing_files.rb",
+     "spec/node_spec.rb",
+     "spec/spec.opts",
+     "spec/spec_helper.rb",
+     "spec/tree_spec.rb",
+     "tryme.rb"
+  ]
+  s.homepage = %q{http://github.com/Trevoke/SGFParser}
+  s.rdoc_options = ["--charset=UTF-8"]
+  s.require_paths = ["lib"]
+  s.rubygems_version = %q{1.3.6}
+  s.summary = %q{A library for working with SGF files.}
+  s.test_files = [
+    "spec/node_spec.rb",
+     "spec/spec_helper.rb",
+     "spec/tree_parser_spec.rb",
+     "spec/tree_spec.rb"
+  ]
+
+  if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
+      s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
+    else
+      s.add_dependency(%q<rspec>, [">= 1.2.9"])
+    end
+  else
+    s.add_dependency(%q<rspec>, [">= 1.2.9"])
+  end
+end
+

data/VERSION

@@ -1 +1 @@
-0.8.0
+0.9.0

data/lib/sgf/parser/node.rb

@@ -23,8 +23,11 @@ module SgfParser
     # list or an array of node children. Will raise an error if one of the
     # arguments is not of class Node.
     def add_children *nodes
+      nodes.flatten!
       raise "Non-node child given!" if nodes.find { |node| node.class != Node }
-      @children.concat nodes.flatten
+      # This node becomes the proud parent of one or more node!
+      nodes.each { |node| node.parent = self }
+      @children.concat nodes
     end
 
     # Adds one or more properties to the node.

data/lib/sgf/parser/tree.rb

@@ -13,20 +13,19 @@ module SgfParser
     # options: \n
     # :filename => filename \n
     # !!! OR !!! \n
-    # :sgf_string => string \n
+    # :string => string \n
     def initialize args={}
       @root = Node.new
       @sgf = ""
-      raise ArgumentError, "Both file and string provided" if args[:filename] &&
-                                                              args[:sgf_string]
-      if !args[:filename].nil?
+      raise ArgumentError, "Both file and string provided" if args[:filename] && args[:string]
+      if args[:filename]
         load_file args[:filename]
-      elsif !args[:sgf_string].nil?
-        load_string args[:sgf_string]
+      elsif args[:string]
+        load_string args[:string]
       end
 
-    end # initialize
-
+      parse unless @sgf.empty?
+    end
 
     # Iterates over the tree, node by node, in preorder fashion.
     # Does not support other types of iteration, but may in the future.
@@ -36,70 +35,65 @@ module SgfParser
         when :preorder
           preorder @root, &block
       end
-    end # each
+    end
 
     # Compares a tree to another tree, node by node.
-    # Nodes must by the same (same properties, parents and children).
+    # Nodes must be the same (same properties, parents and children).
     def == other_tree
       one = []
       two = []
       each { |node| one << node }
       other_tree.each { |node| two << node }
       one == two
-    end # ==
+    end
 
     # Saves the tree as an SGF file. raises an error if a filename is not given.
     # tree.save :filename => file_name
     def save args={}
       raise ArgumentError, "No file name provided" if args[:filename].nil?
       # SGF files are trees stored in pre-order traversal.
-      @sgf_string = "("
+      @savable_sgf = "("
       @root.children.each { |child| write_node child }
       # write_node @root
-      @sgf_string << ")"
+      @savable_sgf << ")"
 
-      File.open(args[:filename], 'w') { |f| f << @sgf_string }
-    end #save
+      File.open(args[:filename], 'w') { |f| f << @savable_sgf }
+    end
 
     private
 
-    # Adds a stringified node to the variable @sgf_string - for saving purposes. 
+    # Adds a stringified node to the variable @savable_sgf.
     def write_node node=@root
-      @sgf_string << ";"
+      @savable_sgf << ";"
       unless node.properties.empty?
         properties = ""
         node.properties.each do |k, v|
           v_escaped = v.gsub("]", "\\]")
           properties += "#{k.to_s}[#{v_escaped}]"
         end
-        @sgf_string << "#{properties}"
+        @savable_sgf << "#{properties}"
       end
 
       case node.children.size
         when 0
-          @sgf_string << ")"
+          @savable_sgf << ")"
         when 1
           write_node node.children[0]
         else
           node.each_child do |child|
-            @sgf_string << "("
+            @savable_sgf << "("
             write_node child
           end
       end
     end
 
-    # Used to load and parse a string if a string was given.
     def load_string string
       @sgf = string
-      parse unless @sgf.empty?
-    end # load_string
+    end
 
-    # Used to load and parse a file if a file was given.
     def load_file filename
-      @sgf = ""
       File.open(filename, 'r') { |f| @sgf = f.read }
-      parse unless @sgf.empty?
-    end # load_file
+    end 
 
     # Traverse the tree in preorder fashion, starting with the @root node if
     # no node is given, and activating the passed block on each.

data/lib/sgf/parser/tree_parse.rb

@@ -1,111 +1,97 @@
+require 'stringio'
+
 module SgfParser
   class Tree
 
     private
 
-    # This function parses a SGF string into a linked list, or tree.
+    # Creates a tree (truly, a linked list) from @sgf.
     def parse
+      while char = next_character
+        case char
+          when '(' then store_branch
+          when ')' then fetch_branch
+          when ';' then store_node_and_create_new_node
+          when '[' then get_and_store_property
+          else store_character(char)
+        end
+      end
+    end
+
+    def next_character
+      character_available? && @stream.sysread(1)
+    end
+
+    def character_available?
+      @stream ||= StringIO.new clean_string, 'r'
+      !@stream.eof?
+    end
+
+    def clean_string
       @sgf.gsub! "\\\\n\\\\r", ""
       @sgf.gsub! "\\\\r\\\\n", ""
       @sgf.gsub! "\\\\r", ""
       @sgf.gsub! "\\\\n", ""
-      #@sgf.gsub! "\n", ""
-      branches = [] # This stores where new branches are open
-      current_node = @root # Let's start at the beginning, shall we?
-      identprop = false # We are not in the middle of an identprop value.
-                        # An identprop is an identity property - a value.
-      content = Hash.new # Hash holding all the properties
-      param, property = "", "" # Variables holding the idents and props
-      end_of_a_series = false # To keep track of params with multiple properties
-
-      sgf_array = @sgf.split(//)
-      iterator = 0
-      array_length = sgf_array.size
-
-      while iterator < array_length
-        char = sgf_array[iterator]
-        case char
-=begin
-Basically, if we're inside an identprop, it's a normal character (or a closing
-character).
-If we're not, it's either a property or a special SGF character so we have
-to handle that properly.
-=end
-          when '(' # Opening a new branch
-            if identprop
-              property << char
-            else
-              branches.unshift current_node
-            end
-          when ')' # Closing a branch
-            if identprop
-              property << char
-            else
-              current_node = branches.shift
-              param, property = "", ""
-              content.clear              
-            end
-          when ';' # Opening a new node
-            if identprop
-              property << char
-            else
-              # Make the current node the old node, make new node, store data
-                parent = current_node
-                current_node = Node.new :parent => parent
-                parent.add_properties content
-                parent.add_children current_node
-                param, property = "", ""
-                content.clear
-            end
-          when '[' # Open identprop?
-            if identprop
-              property << char
-            else # If we're not inside an identprop, then now we are.
-              identprop = true
-              end_of_a_series = false
-            end
-          when ']' # Close identprop
-# Cleverness : checking for this first, then for the backslash.
-# This means that if we encounter this, it must be closing an identprop.
-# Because the "\\" code handles the logic to see if we're inside an identprop,
-# And for skipping the bracket if necessary.            
-            end_of_a_series = true # Maybe end of a series of identprop.
-            identprop = false # That's our cue to close an identprop.
-            content[param] = property
-            property = ""
-          when "\\"
-            # If we're inside a comment, then maybe we're about to escape a ].
-            # This is the whole reason we need this ugly loop.
-            if identprop
-              # If the next character is a closing bracket, then it's just
-              # escaped and the identprop continues.
-              if sgf_array[iterator + 1] == "]"
-                property << "]"
-                # On the next pass through, we will skip that bracket.
-                iterator += 1
-              else
-                property << "\\"
-              end
-            else
-              #This should never happen - a backslash outside an identprop ?!
-              #But let's not have it be told that I'm not prepared.
-              param << "\\"
-            end
-          when "\n"
-            property << "\n" if identprop
-
-          else
-            # Well, I guess it's "just" a character after all.
-            if end_of_a_series
-              end_of_a_series = false
-              param, property = "", ""
-            end
-            identprop ? (property << char) : param << char
-        end
-        iterator += 1
+      @sgf
+    end    
+
+    def store_branch
+      @branches ||= []
+      @branches.unshift @current_node
+    end
+
+    def current_node
+      @current_node ||= @root
+    end    
+
+    def fetch_branch
+      @current_node = @branches.shift
+      clear_temporary_data
+    end
+
+    def store_node_and_create_new_node
+      parent = current_node
+      @current_node = Node.new :parent => parent
+      parent.add_properties content
+      parent.add_children @current_node
+      clear_temporary_data
+    end
+
+    def get_and_store_property
+      @content[@identity] ||= ""
+      @content[@identity] << get_property
+      @identity = ""
+    end
+
+    def get_property
+      buffer = ""
+      while true
+        next_bit = @stream.sysread(1)
+        break if next_bit == "]"
+        next_bit << @stream.sysread(1) if next_bit == "\\"
+        next_bit = "]" if next_bit == "\\]"
+        buffer << next_bit
       end
+      buffer
+    end
 
+    def store_character(char)
+      @identity << char unless char == "\n"
+    end
+
+    def clear_temporary_data
+      @content.clear
+      @identity = ""
+    end
+
+    def content
+      @content ||= {}
+    end
+
+    def identity
+      @identity ||= ""      
     end
 
   end
-end
+end
+

data/lib/sgf/sgf_indent.rb

@@ -0,0 +1,108 @@
+require 'stringio'
+
+module SgfParser
+
+  # This indents an SGF file to make it more readable. It outputs to the screen
+  # by default, but can be given a file as output.
+  # Usage: SgfParser::Indenter.new infile, outfile
+  class Indenter
+
+    def initialize file, out=$stdout
+      @stream = load_file_to_stream(file)
+      @new_string = ""
+      @indentation = 0
+      @out = (out == $stdout) ? $stdout : File.open(out, 'w')
+      parse
+    end
+
+    def next_character
+      !@stream.eof? && @stream.sysread(1)
+    end
+
+    def parse
+      while char = next_character
+        case char
+          when '(' then open_branch
+          when ')' then close_branch
+          when ';' then new_node
+          when '[' then add_property
+          else add_identity(char)
+        end
+      end
+
+      @out << @new_string
+      @out.close unless @out == $stdout
+      #if out == $stdout
+      #  $stdout << @new_string
+      #else
+      #  File.open(out, 'w') { |file| file << @new_string }
+      #end      
+    end
+
+    def open_branch
+      next_line
+      @indentation += 2
+      @new_string << " " * @indentation
+      @new_string << "("
+    end
+
+    def close_branch
+      @new_string << ")"
+      next_line
+      @indentation -= 2
+      @indentation = 0 if @indentation < 0
+      @new_string << " " * @indentation
+    end
+
+    def new_node
+      next_line
+      @new_string << " " * @indentation
+      @new_string << ";"
+    end
+
+    def next_line
+      @new_string << "\n"
+    end
+
+    #TODO Fix it more. Add _ONE_ set of indentation if there are newlines,
+    #TODO Not one set for every newline.
+    def add_property
+      buffer = "["
+      while true
+        next_bit = @stream.sysread(1)
+        next_bit << @stream.sysread(1) if next_bit == "\\"
+        buffer << next_bit
+        buffer << " " * @indentation if next_bit == "\n"
+        break if next_bit == "]"
+      end
+      buffer << "\n"
+      buffer << " " * @indentation
+      @new_string << buffer
+    end
+
+    def add_identity(char)
+      @new_string << char unless char == "\n"
+    end
+
+    private
+
+    def load_file_to_stream(file)
+      sgf = ""
+      File.open(file) { |f| sgf = f.read }
+      clean_string(sgf)
+      return StringIO.new(sgf, 'r')
+    end
+
+    def clean_string(sgf)
+      sgf.gsub! "\\\\n\\\\r", ""
+      sgf.gsub! "\\\\r\\\\n", ""
+      sgf.gsub! "\\\\r", ""
+      sgf.gsub! "\\\\n", ""
+    end    
+
+  end
+end
+
+
+
+

data/lib/sgf_parser.rb

@@ -5,4 +5,4 @@ require 'sgf/parser/properties'
 require 'sgf/parser/node'
 require 'sgf/parser/tree'
 require 'sgf/parser/tree_parse'
-require 'sgf/sgfindent'
+require 'sgf/sgf_indent'

data/spec/tree_parser_spec.rb

@@ -0,0 +1,13 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe "SgfParser::Tree.parse" do
+
+  before :each do
+    @tree = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf'
+  end
+
+  it "should return true" do
+    true
+  end
+
+end

data/spec/tree_spec.rb

@@ -16,7 +16,6 @@ describe "SgfParser::Tree" do
     @new = 'sample_sgf/ff4_ex_saved.sgf'
     @tree.save :filename => @new
     @tree2 = SgfParser::Tree.new :filename => @new
-    
     @tree2.should == @tree
   end
 

data/tryme.rb

@@ -0,0 +1,3 @@
+require 'irb/completion'
+require 'lib/sgf_parser'
+sgf = SgfParser::Tree.new :filename => 'sample_sgf/ff4_ex.sgf' 

metadata

@@ -1,7 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: SgfParser
 version: !ruby/object:Gem::Version 
-  version: 0.8.0
+  prerelease: false
+  segments: 
+  - 0
+  - 9
+  - 0
+  version: 0.9.0
 platform: ruby
 authors: 
 - Aldric Giacomoni
@@ -9,19 +14,23 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-01-05 00:00:00 -05:00
+date: 2010-04-30 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rspec
-  type: :development
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        segments: 
+        - 1
+        - 2
+        - 9
         version: 1.2.9
-    version: 
+  type: :development
+  version_requirements: *id001
 description: SGFParser is a library that parses and saves SGF (Smart Game Format) files.
 email: aldric@trevoke.net
 executables: []
@@ -43,16 +52,16 @@ files:
 - lib/sgf/parser/properties.rb
 - lib/sgf/parser/tree.rb
 - lib/sgf/parser/tree_parse.rb
-- lib/sgf/sgfindent.rb
+- lib/sgf/sgf_indent.rb
 - lib/sgf_parser.rb
 - sample_sgf/ff4_ex.sgf
-- sample_sgf/ff4_ex_saved.sgf
 - sample_sgf/redrose-tartrate.sgf
 - sample_usage/parsing_files.rb
 - spec/node_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb
 - spec/tree_spec.rb
+- tryme.rb
 has_rdoc: true
 homepage: http://github.com/Trevoke/SGFParser
 licenses: []
@@ -66,22 +75,25 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      segments: 
+      - 0
       version: "0"
-  version: 
 required_rubygems_version: !ruby/object:Gem::Requirement 
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      segments: 
+      - 0
       version: "0"
-  version: 
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.5
+rubygems_version: 1.3.6
 signing_key: 
 specification_version: 3
 summary: A library for working with SGF files.
 test_files: 
 - spec/node_spec.rb
 - spec/spec_helper.rb
+- spec/tree_parser_spec.rb
 - spec/tree_spec.rb

data/lib/sgf/sgfindent.rb

@@ -1,118 +0,0 @@
-module SgfParser
-
-  # This indents an SGF file to make it more readable. It outputs to the screen
-  # by default, but can be given a file as output.
-  # Usage: SgfParser::Indenter.new infile, outfile
-  class Indenter
-
-    def initialize file, out=$stdout
-      sgf = ""
-      File.open(file) { |f| sgf = f.read }
-      @new_string = ""
-      sgf.gsub! "\\\\n\\\\r", ""
-      sgf.gsub! "\\\\r\\\\n", ""
-      sgf.gsub! "\\\\r", ""
-      sgf.gsub! "\\\\n", ""
-      #sgf.gsub! "\n", ""
-
-      end_of_a_series = false
-      identprop = false # We are not in the middle of an identprop value.
-      # An identprop is an identity property - a value.
-
-      sgf_array = sgf.split(//)
-      iterator = 0
-      array_length = sgf_array.size
-      indent = 0
-
-      while iterator < array_length - 1
-        char = sgf_array[iterator]
-
-        case char
-          when '(' # Opening a new branch
-            if !identprop
-              @new_string << "\n"
-              indent += 2
-              #tabulate indent
-              @new_string << " " * indent
-            end
-            @new_string << char
-
-          when ')' # Closing a branch
-            @new_string << char
-            if !identprop
-              @new_string << "\n"
-              indent -= 2
-              @new_string << " " * indent
-              #tabulate indent
-            end
-
-          when ';' # Opening a new node
-            if !identprop
-              @new_string << "\n"
-              @new_string << " " * indent
-              #tabulate indent
-            end
-            @new_string << char
-
-          when '[' # Open comment?
-            if !identprop #If we're not inside a comment, then now we are.
-              identprop = true
-              end_of_a_series = false
-            end
-            @new_string << char
-
-          when ']' # Close comment
-            end_of_a_series = true # Maybe end of a series of comments.
-            identprop = false # That's our cue to close a comment.
-            @new_string << char
-
-          when "\\" # If we're inside a comment, then maybe we're about to escape a ].
-            # This is the whole reason we need this ugly charade of a loop.
-            if identprop
-              if sgf_array[iterator + 1] == "]"
-                @new_string << "\\]"
-                iterator += 1
-              else
-                @new_string << "\\"
-              end
-            else
-              #This should never happen - a backslash outside a comment ?!
-              #But let's not have it be told that I'm not prepared.
-              @new_string << "\\"
-            end
-
-          when "\n"
-            @new_string << "\n"
-            @new_string << " " * indent
-            #tabulate indent
-
-          else
-            # Well, I guess it's "just" a character after all.
-            if end_of_a_series
-              end_of_a_series = false
-            end
-            @new_string << char
-        end
-
-        iterator += 1
-      end
-
-      if out == $stdout
-        $stdout << @new_string
-      else
-        File.open(out, 'w') { |file| file << @new_string }
-      end
-      
-    end
-
-    private
-
-    def tabulate indent
-      indent.times { print " " }
-    end
-  end
-end
-
-
-
-

data/sample_sgf/ff4_ex_saved.sgf

@@ -1,45 +0,0 @@
-(;FF[4]AP[Primiview:3.1]GM[1]SZ[19]GN[Gametree 1: properties]US[Arno Hollosi](;B[pd]N[Moves, comments, annotations]C[Nodename set to: "Moves, comments, annotations"];W[dp]C[Marked as "Good for White"]GW[1];B[pp]C[Marked as "Very good for Black"]GB[2];W[dc]C[Marked as "Very good for White"]GW[2];B[pj]C[Marked as "Even position"]DM[1];W[ci]UC[1]C[Marked as "Unclear position"];B[jd]C[Marked as "Tesuji" or "Good move"]TE[1];BM[2]W[jp]C[Marked as "Very bad move"];DO[]B[gd]C[Marked as "Doubtful move"];IT[]W[de]C[Marked as "Interesting move"];B[jj];W[]C[White "Pass" move];)(;AB[do:gq]N[Setup]C[Black & white stones at the top are added as single stones.
-
-Black & white stones at the bottom are added using compressed point lists.]  AW[pn:pq];C[AddEmpty
-
-Black stones & stones of left white group are erased in FF[3\] way.
-
-White stones at bottom right were erased using compressed point list.]AE[pn:pq];AW[pp]AB[pd]C[Added two stones.
-
-Node marked with "Black to play".]PL[B];)(;AW[er]AB[mr]N[Markup]C[Position set up without compressed point lists.]; SL[kh:kj] TW[lr:ns] TB[ao]C[Markup at top partially using compressed point lists (for markup on white stones); listed clockwise, starting at upper left:
-- TR (triangle)
-- CR (circle)
-- SQ (square)
-- SL (selected points)
-- MA ('X')
-
-Markup at bottom: black & white territory (using compressed point lists)] CR[pd:pf]TR[fd:ff] SQ[ph:pj] MA[fh:fj];LB[ms:12345678]C[Label (LB property)
-
-Top: 8 single char labels (1-4, a-d)
-
-Bottom: Labels up to 8 char length.];)(;B[qd]N[Style & text type]C[There are hard linebreaks & soft linebreaks.
-Soft linebreaks are linebreaks preceeded by '\\' like this one >o\
-k<. Hard line breaks are all other linebreaks.
-Soft linebreaks are converted to >nothing<, i.e. removed.
-
-Note that linebreaks are coded differently on different systems.
-
-Examples (>ok< shouldn't be split):
-
-linebreak 1 "": >o\
-k<
-linebreak 2 "": >o\
-
-linebreak 3 "": >o\
-k<
-linebreak 4 "": >o\
-Black stones left (in this byo-yomi period): 10]OB[10]BL[105.6];W[qq]C[White time left: 200 sec
-White stones left: 2]OW[2]WL[200];B[sr]C[Black time left: 87 sec
-Black stones left: 9]OB[9]BL[87.00];W[qs]C[White time left: 13.2 sec
-White stones left: 1]OW[1]WL[13.20];B[rs]C[One white stone at s2 captured];W[ps];B[pr];W[or]MN[2]C[Set move number to 2];B[os]C[Two white stones captured
-(at q1 & r1)];MN[112]W[pq]C[Set move number to 112];B[sq];W[rp];B[ps];W[ns];B[ss];W[nr];B[rr];W[sp];);FF[4]C[Gametree 2: game-info
-
-Game-info properties are usually stored in the root node.
-If games are merged into a single game-tree, they are stored in the node\
- where the game first becomes distinguishable from all other games in\
- the tree.]AP[Primiview:3.1]GM[1]SZ[19];B[pd](;)(;)(;W[ep];B[pp](;)(;))