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.

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

AI - Livestock escape fixed, water jumping fixed #1947

Closed
wants to merge 5 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@SafwatHalaby
Member

SafwatHalaby commented May 6, 2015

No description provided.

@SafwatHalaby

This comment has been minimized.

Show comment
Hide comment
@SafwatHalaby

SafwatHalaby May 6, 2015

Member

With the exception of bool cMonster::EnsureProperDestination(cChunk & a_Chunk), the changes are trivial.

I did my best to not misuse chunks in that function, but can anyone double-check that?

Member

SafwatHalaby commented May 6, 2015

With the exception of bool cMonster::EnsureProperDestination(cChunk & a_Chunk), the changes are trivial.

I did my best to not misuse chunks in that function, but can anyone double-check that?

@madmaxoft

View changes

Show outdated Hide outdated src/Mobs/Monster.cpp
@madmaxoft

View changes

Show outdated Hide outdated src/Mobs/Monster.cpp
@madmaxoft

View changes

Show outdated Hide outdated src/Mobs/Monster.cpp
@madmaxoft

View changes

Show outdated Hide outdated src/Mobs/Monster.h
wiseoldman95
@madmaxoft

This comment has been minimized.

Show comment
Hide comment
@madmaxoft

madmaxoft May 6, 2015

Member

I'd have preferred a doxy-comment without empty lines, but an empty lines between the previous function and the next function's doxy-comment.

Member

madmaxoft commented May 6, 2015

I'd have preferred a doxy-comment without empty lines, but an empty lines between the previous function and the next function's doxy-comment.

wiseoldman95
@SafwatHalaby

This comment has been minimized.

Show comment
Hide comment
@SafwatHalaby

SafwatHalaby May 6, 2015

Member

Done.

Member

SafwatHalaby commented May 6, 2015

Done.

@SafwatHalaby

This comment has been minimized.

Show comment
Hide comment
@SafwatHalaby

SafwatHalaby May 6, 2015

Member

Appveyor finished compiling way back but it's stuck. It might have to do with the Github downtime that occured. Nevertheless, all CI's are successful.

Member

SafwatHalaby commented May 6, 2015

Appveyor finished compiling way back but it's stuck. It might have to do with the Github downtime that occured. Nevertheless, all CI's are successful.

@SafwatHalaby

This comment has been minimized.

Show comment
Hide comment
@SafwatHalaby

SafwatHalaby May 6, 2015

Member

Rebased to shake the CI.

Member

SafwatHalaby commented May 6, 2015

Rebased to shake the CI.

@SafwatHalaby

This comment has been minimized.

Show comment
Hide comment
@SafwatHalaby

SafwatHalaby May 6, 2015

Member

Things are acting weird, re-opening a new branch.

Member

SafwatHalaby commented May 6, 2015

Things are acting weird, re-opening a new branch.

@SafwatHalaby

This comment has been minimized.

Show comment
Hide comment
@SafwatHalaby
Member

SafwatHalaby commented May 6, 2015

@SafwatHalaby SafwatHalaby deleted the SafwatHalaby:jumps branch May 8, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment