Skip to content

Commit

Permalink
fix compatibility with new compiler constrainst
Browse files Browse the repository at this point in the history
  • Loading branch information
benoist committed Apr 13, 2016
1 parent 95d4fb2 commit 51d379a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/message_pack/lexer.cr
Expand Up @@ -14,7 +14,7 @@ class MessagePack::Lexer
@io = io
@token = Token.new
@byte_number = 0
@current_byte = 0
@current_byte = 0_u8
@eof = false
end

Expand Down
3 changes: 3 additions & 0 deletions src/message_pack/token.cr
Expand Up @@ -12,6 +12,9 @@ class MessagePack::Token
property :used

@size : Int64
@int_value : Int::Signed
@uint_value : Int::Unsigned
@float_value : Float32 | Float64

def initialize
@type = :EOF
Expand Down
2 changes: 1 addition & 1 deletion src/message_pack/unpack_exception.cr
@@ -1,7 +1,7 @@
# Raised on invalid MessagePack strings.
class MessagePack::UnpackException < Exception
# The line number where the invalid MessagePack was detected.
getter byte_number
getter byte_number : Int32

# Creates a ParseException with the given message and byte number.
def initialize(message, @byte_number)
Expand Down

2 comments on commit 51d379a

@asterite
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@benoist Wow. Only those changes were needed to upgrade to the new version?

@benoist
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes that was it! :-)

Please sign in to comment.