search

wincent / command-t

⌨️ Fast file navigation for Neovim and Vim
BSD 2-Clause "Simplified" License
2.74k stars 317 forks source link

Fix RangeError in NUM2LONG conversion on Windows x64 #304

Closed arkeet closed 7 years ago

arkeet commented 7 years ago

I was hitting an issue on 64-bit Windows where on every search it would hit RangeError: bignum too big to convert into `long'. This error was due to some NUM2LONG conversions - I suspect it's because on 64-bit Windows, long (32 bits) is too small to fit a pointer (64 bits)? I think it's enough to compare object ids directly with rb_equal, and doing so fixes the error for me.

wincent commented 7 years ago

Thanks @arkeet. Seems reasonable to me.