Skip to content
This repository has been archived by the owner on May 23, 2021. It is now read-only.

fix: add todo puzzles #5

Merged
merged 1 commit into from
Dec 26, 2020
Merged

fix: add todo puzzles #5

merged 1 commit into from
Dec 26, 2020

Conversation

olchyk98
Copy link
Owner

No description provided.

@olchyk98 olchyk98 self-assigned this Dec 26, 2020
@todo
Copy link

todo bot commented Dec 26, 2020

Optimize

runStart = x + 1; // @todo Optimize
} else if (x + 1 < _mazeWidth)
{
_grid[y, x].borderedE = false;


This comment was generated by todo based on a todo comment in 337a37a in #5. cc @olchyk98.

@todo
Copy link

todo bot commented Dec 26, 2020

Matrix Dimensions should be placed in the inspector.

// @todo Matrix Dimensions should be placed in the inspector.
// WARNING: Minimum value is 5.
private readonly int _matrixWidth = 10;
// WARNING: Minimum value is 5.


This comment was generated by todo based on a todo comment in 337a37a in #5. cc @olchyk98.

@todo
Copy link

todo bot commented Dec 26, 2020

The method call should be memoized. That way we could avoid object referencing or memory leaks.

// @todo The method call should be memoized. That way we could avoid object referencing or memory leaks.
var size = GetCellSize(cell);
cell.transform.position = new Vector3(


This comment was generated by todo based on a todo comment in 337a37a in #5. cc @olchyk98.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant