timholy / Grid.jl

Interpolation and related operations on grids
MIT License
47 stars 26 forks source link

Fix stack overflow in Counter contructor. #10

Closed blakejohnson closed 10 years ago

blakejohnson commented 10 years ago

On julia master, I see that

InterpGrid([0.0], BCnearest, InterpNearest)

causes a stack overflow because of construction of a Counter. Must have been broken by the recent change to convert constructor inputs by default.

I wasn't quite sure what case you were trying to grab with this outer constructor, so I tried the simplest thing that worked.

timholy commented 10 years ago

Thanks a lot! In a surprising coincidence, I had just made that same change in my local branch about 30 minutes after you pushed this, but noticed this before pushing my own change. Interpolation must be in the air...