Skip to content

Boid.pde #94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Boid.pde #94

wants to merge 4 commits into from

Conversation

@mshaub
Copy link

@mshaub mshaub commented Jan 4, 2018

The original file wouldn't run in Processing3 (3.3.2) throwing an error that vairable pos didn't exist. Added the creation of the variable and modified the line throwing the error under display()

@mshaub
Boid.pde
f1a8992 
The original file wouldn't run in Processing3 (3.3.2) throwing an error that vairable pos didn't exist. Added the creation of the variable and modified the line throwing the error under display()
@shiffman
Copy link
Member

shiffman commented Jan 4, 2018

Thanks! Actually the mistake is in the borders() method where loc should be replaced with pos. There also may be more other places where I'm using the nam loc. Do you want to resubmit?

mshaub added 3 commits January 5, 2018 08:40
@mshaub
Boid.pde
6ec4793 
a variable "loc" was used in some locations that should be "pos." All instances of "loc" have been changed to "pos"
@mshaub
Boid
bf54469 
all instances of variable "loc" have been changed to "pos"
@mshaub
Boid.pde
0fe24f1 
All instances of the variable "loc" have been changed to "pos"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mshaub @shiffman