rasulzhan/aforge

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

Closed this issue · 2 comments

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

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

  • Changed state: Fixed
  • Added labels: Priority-Medium, Project-Vision
Great, thanks.

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