Skip to content
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

Fixed bug in F748: Set Position and Motion #170

Merged
merged 2 commits into from
Mar 27, 2017

Conversation

ogmage78
Copy link
Contributor

There has been a bug in this code that was keeping me from being able to drop an item.

  • Fixed alignment problem and cleaned up the ability to report position with the correct flags

  • I moved and expanded an enum flag list for correct scoping

  • I put the position flag in the position object and initialized the flags correctly on player create.

  • Added another signature to Serialize - left the old one for backward compatibility - we may want to look at refactoring this.

Please let me know if you anything that looks wrong. I am still learning. This is more of a plumbing fix. You should not notice anything new or anything new broken :) If this get's merged - I want to start looking at creating mutable objects next.

Copy link
Member

@LtRipley36706 LtRipley36706 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In my testing, this resolved some of the crashes I saw when objects were trying to set velocities (via bitfields)

It did not fix the missing data as Og points out that it just writes 0s

@Mogwai-TheFurry Mogwai-TheFurry merged commit e4f649d into ACEmulator:master Mar 27, 2017
ogmage78 pushed a commit to ogmage78/ACE that referenced this pull request Mar 28, 2017
Created IF statement to remove Welcome popup box

When using this IF statement you can remove the Welcome to this ACE
server!\nFor more information visit http://www.acemulator.org. from
inside the quotes in the config.json file and the popup box does not
appear.

Fixed bug in F748: Set Position and Motion (ACEmulator#170)

* Fixed bug in F748: Set Position and Motion

* Needed to fix the comments

Had to make it match the code correctly.
@ogmage78 ogmage78 deleted the positionwork branch April 14, 2017 19:37
Sign up for free to 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.

None yet

3 participants