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

Update for the new Bukkit event system #1

Merged
merged 4 commits into from
Mar 21, 2012
Merged

Conversation

tyrovc
Copy link

@tyrovc tyrovc commented Mar 21, 2012

Updated event listeners and included events for the new event system.

Since Bukkit events are no longer all Serializable by default, events sent through EventLink need to implement Serializable. This includes marking things from the new event system that should not be Serialized transient, i.e. event handler lists.

@Raphfrk
Copy link
Owner

Raphfrk commented Mar 21, 2012

Thanks, the new event system is pretty annoying.

It is a pity that this means that you can't send standard events to other servers. However, they weren't likely to need that anyway.

I assume you have no problem with your changes being licensed under the MIT license?

https://github.com/Raphfrk/EventLink/blob/master/src/LICENSE

@tyrovc tyrovc closed this Mar 21, 2012
@tyrovc tyrovc reopened this Mar 21, 2012
@tyrovc
Copy link
Author

tyrovc commented Mar 21, 2012

Hit the wrong button on a tiny phone screen.

I don't have any problems with the licensing.

@tyrovc tyrovc closed this Mar 21, 2012
@tyrovc tyrovc reopened this Mar 21, 2012
Raphfrk added a commit that referenced this pull request Mar 21, 2012
Update for the new Bukkit event system
@Raphfrk Raphfrk merged commit e80475f into Raphfrk:master Mar 21, 2012
@Raphfrk
Copy link
Owner

Raphfrk commented Mar 21, 2012

Btw, are you survivals on the forum?

@tyrovc
Copy link
Author

tyrovc commented Mar 21, 2012

Don't know what you mean by survivals. If you're talking about the Bukkit forums, my account is TyrOvC.

@Raphfrk
Copy link
Owner

Raphfrk commented Mar 22, 2012

It is a username, he posted edits in the thread, thought you might be him.

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.

2 participants