Change nonsense return types to Nil: JSON and YAML

src/big/json.cr

@@ -3,7 +3,7 @@ require "big"
 
 class JSON::Builder
   # Writes a big decimal.
-  def number(number : BigDecimal) : IO
+  def number(number : BigDecimal) : Nil
     scalar do
       @io << number
     end
@@ -26,7 +26,7 @@ struct BigInt
     to_s
   end
 
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.number(self)
   end
 end
@@ -56,7 +56,7 @@ struct BigFloat
     to_s
   end
 
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.number(self)
   end
 end
@@ -86,7 +86,7 @@ struct BigDecimal
     to_s
   end
 
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.number(self)
   end
 end

src/json/builder.cr

@@ -28,7 +28,7 @@ class JSON::Builder
   end
 
   # Starts a document.
-  def start_document : JSON::Builder::DocumentStartState
+  def start_document : Nil
     case state = @state.last
     when StartState
       @state[-1] = DocumentStartState.new
@@ -62,28 +62,28 @@ class JSON::Builder
   end
 
   # Writes a `null` value.
-  def null : IO
+  def null : Nil
     scalar do
       @io << "null"
     end
   end
 
   # Writes a boolean value.
-  def bool(value : Bool) : IO
+  def bool(value : Bool) : Nil
     scalar do
       @io << value
     end
   end
 
   # Writes an integer.
-  def number(number : Int) : IO
+  def number(number : Int) : Nil
     scalar do
       @io << number
     end
   end
 
   # Writes a float.
-  def number(number : Float) : IO
+  def number(number : Float) : Nil
     scalar do
       case number
       when .nan?
@@ -100,7 +100,7 @@ class JSON::Builder
   # by invoking `to_s` on it.
   #
   # This method can also be used to write the name of an object field.
-  def string(value) : IO
+  def string(value) : Nil
     string = value.to_s
 
     scalar(string: true) do
@@ -157,22 +157,22 @@ class JSON::Builder
   # the IO without processing. This is the only method that
   # might lead to invalid JSON being generated, so you must
   # be sure that *string* contains a valid JSON string.
-  def raw(string : String) : IO
+  def raw(string : String) : Nil
     scalar do
       @io << string
     end
   end
 
   # Writes the start of an array.
-  def start_array : IO
+  def start_array : Nil
     start_scalar
     increase_indent
     @state.push ArrayState.new(empty: true)
     @io << '['
   end
 
   # Writes the end of an array.
-  def end_array : JSON::Builder::ArrayState | JSON::Builder::DocumentEndState | JSON::Builder::ObjectState
+  def end_array : Nil
     case state = @state.last
     when ArrayState
       @state.pop
@@ -193,15 +193,15 @@ class JSON::Builder
   end
 
   # Writes the start of an object.
-  def start_object : IO
+  def start_object : Nil
     start_scalar
     increase_indent
     @state.push ObjectState.new(empty: true, name: true)
     @io << '{'
   end
 
   # Writes the end of an object.
-  def end_object : JSON::Builder::ArrayState | JSON::Builder::DocumentEndState | JSON::Builder::ObjectState
+  def end_object : Nil
     case state = @state.last
     when ObjectState
       unless state.name
@@ -235,12 +235,12 @@ class JSON::Builder
   end
 
   # :ditto:
-  def scalar(value : Int | Float) : IO
+  def scalar(value : Int | Float) : Nil
     number(value)
   end
 
   # :ditto:
-  def scalar(value : String) : IO
+  def scalar(value : String) : Nil
     string(value)
   end
 

src/json/pull_parser.cr

@@ -263,7 +263,7 @@ class JSON::PullParser
   # Reads the new value and fill the a JSON builder with it.
   #
   # Use this method with a `JSON::Builder` to read a JSON while building another one.
-  def read_raw(json) : IO | JSON::PullParser::Kind
+  def read_raw(json) : Nil
     case @kind
     when .null?
       read_next
@@ -556,7 +556,7 @@ class JSON::PullParser
   #
   # It skips the whole value, not only the next lexer's token.
   # For example if the next value is an array, the whole array will be skipped.
-  def skip : Bool
+  def skip : Nil
     @lexer.skip = true
     skip_internal
     @lexer.skip = false

src/json/to_json.cr

@@ -25,7 +25,7 @@ class Object
 end
 
 struct Nil
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.null
   end
 
@@ -35,13 +35,13 @@ struct Nil
 end
 
 struct Bool
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.bool(self)
   end
 end
 
 struct Int
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.number(self)
   end
 
@@ -51,7 +51,7 @@ struct Int
 end
 
 struct Float
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.number(self)
   end
 
@@ -61,7 +61,7 @@ struct Float
 end
 
 class String
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.string(self)
   end
 
@@ -71,7 +71,7 @@ class String
 end
 
 struct Path
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     @name.to_json(json)
   end
 
@@ -81,7 +81,7 @@ struct Path
 end
 
 struct Symbol
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.string(to_s)
   end
 
@@ -132,7 +132,7 @@ class Hash
 end
 
 struct Tuple
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.array do
       {% for i in 0...T.size %}
         self[{{i}}].to_json(json)
@@ -154,7 +154,7 @@ struct NamedTuple
 end
 
 struct Time::Format
-  def to_json(value : Time, json : JSON::Builder) : IO
+  def to_json(value : Time, json : JSON::Builder) : Nil
     format(value).to_json(json)
   end
 end
@@ -263,7 +263,7 @@ struct Time
   # a time value.
   #
   # See `#from_json` for reference.
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.string(Time::Format::RFC_3339.format(self, fraction_digits: 0))
   end
 end
@@ -343,7 +343,7 @@ end
 # person.to_json    # => %({"birth_date":1459859781})
 # ```
 module Time::EpochConverter
-  def self.to_json(value : Time, json : JSON::Builder) : IO
+  def self.to_json(value : Time, json : JSON::Builder) : Nil
     json.number(value.to_unix)
   end
 end
@@ -367,7 +367,7 @@ end
 # timestamp.to_json # => %({"value":1459860483856})
 # ```
 module Time::EpochMillisConverter
-  def self.to_json(value : Time, json : JSON::Builder) : IO
+  def self.to_json(value : Time, json : JSON::Builder) : Nil
     json.number(value.to_unix_ms)
   end
 end
@@ -394,7 +394,7 @@ end
 # raw.to_json # => %({"value":123456789876543212345678987654321})
 # ```
 module String::RawConverter
-  def self.to_json(value : String, json : JSON::Builder) : IO
+  def self.to_json(value : String, json : JSON::Builder) : Nil
     json.raw(value)
   end
 end

src/oauth2/access_token/bearer.cr

@@ -13,7 +13,7 @@ class OAuth2::AccessToken::Bearer < OAuth2::AccessToken
     request.headers["Authorization"] = "Bearer #{access_token}"
   end
 
-  def to_json(json : JSON::Builder) : IO?
+  def to_json(json : JSON::Builder) : Nil
     json.object do
       json.field "token_type", "bearer"
       json.field "access_token", access_token

src/oauth2/access_token/mac.cr

@@ -45,7 +45,7 @@ class OAuth2::AccessToken::Mac < OAuth2::AccessToken
     Base64.strict_encode OpenSSL::HMAC.digest(digest, mac_key, normalized_request_string)
   end
 
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.object do
       json.field "token_type", "mac"
       json.field "access_token", access_token

src/uuid/json.cr

@@ -32,7 +32,7 @@ struct UUID
   # uuid = UUID.new("87b3042b-9b9a-41b7-8b15-a93d3f17025e")
   # uuid.to_json # => "\"87b3042b-9b9a-41b7-8b15-a93d3f17025e\""
   # ```
-  def to_json(json : JSON::Builder) : IO
+  def to_json(json : JSON::Builder) : Nil
     json.string(to_s)
   end
 

src/yaml/any.cr

@@ -311,7 +311,7 @@ struct YAML::Any
     raw.to_yaml(io)
   end
 
-  def to_json(builder : JSON::Builder) : IO?
+  def to_json(builder : JSON::Builder) : Nil
     if (raw = self.raw).is_a?(Slice)
       raise "Can't serialize #{raw.class} to JSON"
     else

src/yaml/builder.cr

@@ -62,7 +62,7 @@ class YAML::Builder
   end
 
   # Ends a YAML stream.
-  def end_stream : IO | Int32 | Nil
+  def end_stream : Nil
     emit stream_end
     @io.flush
   end
@@ -104,7 +104,7 @@ class YAML::Builder
   end
 
   # Ends a sequence.
-  def end_sequence : Int32
+  def end_sequence : Nil
     emit sequence_end
     decrease_nesting
   end
@@ -123,7 +123,7 @@ class YAML::Builder
   end
 
   # Ends a mapping.
-  def end_mapping : Int32
+  def end_mapping : Nil
     emit mapping_end
     decrease_nesting
   end
@@ -186,7 +186,7 @@ class YAML::Builder
   end
 
   # Closes the builder, freeing up resources.
-  def close : Bool
+  def close : Nil
     finalize
     @closed = true
   end

src/yaml/nodes/parser.cr

@@ -46,7 +46,7 @@ class YAML::Nodes::Parser < YAML::Parser
     node.start_column = @pull_parser.start_column.to_i
   end
 
-  def end_value(node) : Int32
+  def end_value(node) : Nil
     node.end_line = @pull_parser.end_line.to_i
     node.end_column = @pull_parser.end_column.to_i
   end
@@ -72,15 +72,15 @@ class YAML::Nodes::Parser < YAML::Parser
     documents << document
   end
 
-  def add_to_document(document, node) : Array(YAML::Nodes::Node)
+  def add_to_document(document, node) : Nil
     document << node
   end
 
-  def add_to_sequence(sequence, node) : Array(YAML::Nodes::Node)
+  def add_to_sequence(sequence, node) : Nil
     sequence << node
   end
 
-  def add_to_mapping(mapping, key, value) : Array(YAML::Nodes::Node)
+  def add_to_mapping(mapping, key, value) : Nil
     mapping[key] = value
   end
 end

src/yaml/parser.cr

@@ -157,7 +157,7 @@ abstract class YAML::Parser
   end
 
   # Closes this parser, freeing up resources.
-  def close : Bool
+  def close : Nil
     @pull_parser.close
   end
 

src/yaml/pull_parser.cr

@@ -321,7 +321,7 @@ class YAML::PullParser
     LibYAML.yaml_event_delete(pointerof(@event))
   end
 
-  def close : Bool
+  def close : Nil
     finalize
     @closed = true
   end