samples/simple-sphere.rb crashes on 64 bit systems

[shoes]

samples/simple-sphere.rb crashes on 64 bit systems

[ccoupe]

The Shoes manual says nothing about the blur and some other transforms.

[IanTrudel]

Could not duplicate the issue on Windows 8.1, 64 bit, Shoes 3.2.19. Even changing any or all blur to large values (over 100) do not cause Shoes to crash.

[IanTrudel]

Duplicated the issue on MacOS X 10.6.8, Shoes 3.2.19 (my own build). Also confirming changing the value of blur at line 7 from 30 to 25 prevents Shoes from crashing. Console.app provides a backtrace from Ruby.

[ccoupe]

@backorder - the Shoes/Ruby for Windows is 32 bit.

[passenger94]

changing box_blur method in effects.c (adding a cast) works for me
https://github.com/Shoes3/shoes3/blob/master/shoes/effects.c#L71

      out[l2] = run[sums[0]];                        
      out[l2 + 1] = run[sums[1]];                    
      out[l2 + 2] = run[sums[2]];                    
      out[l2 + 3] = run[sums[3]]; 

to

      out[l2] = run[(int)sums[0]];                        
      out[l2 + 1] = run[(int)sums[1]];                    
      out[l2 + 2] = run[(int)sums[2]];                    
      out[l2 + 3] = run[(int)sums[3]]; 

might be related to some changes from 32bits to 64bits OS and the "value" of unsigned char ...
is casting to int the best option here ?

[ccoupe]

@passenger94, that is wonderful! It works for my 64 linux. Please commit the change .

[IanTrudel]

Is this confirmed on MacOS X? If so, let's close the issue.

[ccoupe]

Fixed on OSX as well.