Skip to content

PhaseList: fix run() and unit-tests - #2

Merged
bajtos merged 1 commit into
masterfrom
fix/undefined-phase-var
Nov 6, 2014
Merged

PhaseList: fix run() and unit-tests#2
bajtos merged 1 commit into
masterfrom
fix/undefined-phase-var

Conversation

@bajtos

Copy link
Copy Markdown
Member

/to @ritch please review

@ritch

ritch commented Nov 6, 2014

Copy link
Copy Markdown
Member

Can you change the timeout to 1 ms?
Overall the code looks good. 👍

@bajtos
bajtosforce-pushed the fix/undefined-phase-var branch from bc0a964 to a49c7daCompareNovember 6, 2014 16:46
@bajtos

Copy link
Copy Markdown
MemberAuthor

Can you change the timeout to 1 ms?

Changed, although I don't think the timer is capable of such precision.

bajtos added a commit that referenced this pull request Nov 6, 2014
@bajtos
bajtos merged commit 55c132c into masterNov 6, 2014
@bajtos
bajtos deleted the fix/undefined-phase-var branch November 6, 2014 16:47
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@bajtos@ritch