enricowillemse / aforge

Other
0 stars 0 forks source link

Bug in GridMotionAreaProcessing constructor (wrong parameters for base ctor, line 189) #388

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
GridMotionAreaProcessing(int,int) grid parameters (width, height) are not 
working in this constructor because the base constructor is called twice with 
gridWidth instead of gridWidht / gridHeight.

line 189 in GridMotionAreaProcessing
public GridMotionAreaProcessing ( int gridWidth, int gridHeight ) : this( 
gridWidth, gridWidth, true ) { }

Fix for now:
use different constructor (e.g. GridMotionAreaProcessing ( int gridWidth, int 
gridHeight, bool highlight )

{I know I could fix it myself but I don't have the code checked out and I did 
not yet contribute to the project, so I thought a how-to fix would be faster 
since it will only take 3 seconds to fix}

Original issue reported on code.google.com by i...@davidlindlbauer.com on 4 Sep 2014 at 10:00

GoogleCodeExporter commented 9 years ago
Fixed GridMotionAreaProcessing(int, int) constructor so it correctly forward 
call to another one.

Committed in revision 1732.

Original comment by andrew.k...@gmail.com on 4 Sep 2014 at 10:14

GoogleCodeExporter commented 9 years ago
Great, thanks.

Original comment by i...@davidlindlbauer.com on 4 Sep 2014 at 5:09