gregoriusxu / booksleeve

Automatically exported from code.google.com/p/booksleeve
Other
0 stars 0 forks source link

GetInt64 only supports Int32, not Int64 #30

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Save a value to Redis that will fit in an Int64, but not an Int32.
2. Call connection.Strings.GetInt64(db, key);

What is the expected output? What do you see instead?
Expected the saved value, got an exception instead:
System.OverflowException occurred
  HResult=-2146233066
  Message=Value was either too large or too small for an Int32.
  Source=mscorlib
  StackTrace:
       at System.Number.ParseInt32(String s, NumberStyles style, NumberFormatInfo info)
       at System.Int32.Parse(String s)
       at BookSleeve.RedisResult.get_ValueInt64() in c:\Dev\BookSleeve\BookSleeve\RedisResult.cs:line 36

What version of the product are you using? On what operating system?
NuGet 1.2.0.4

Please provide any additional information below.
I think this is a quick fix to line 36 of RedisResult.cs to change int.Parse to 
Int64.Parse.

Original issue reported on code.google.com by will.tho...@prcm.com on 3 Dec 2012 at 4:06

GoogleCodeExporter commented 8 years ago
Well that's just embarrassing. I will re-deploy within 24 hours.

Original comment by marc.gravell on 3 Dec 2012 at 5:14

GoogleCodeExporter commented 8 years ago
See 1.2.0.5

Original comment by marc.gravell on 3 Dec 2012 at 9:26