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

size checking added #32

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@hohnamkung
Copy link

hohnamkung commented Jul 11, 2013

Pull request for #31

@landakram

This comment has been minimized.

Copy link

landakram commented Nov 6, 2013

Stumbled upon this bug (#31) when trying to use NoSubscriberEvent. Looks like this has been here for 4 months!

Could this please be merged in? It's a one line fix and the NoSubscriberEvent functionality is pretty much broken w/o it. I worked around it, but it would be awesome if it was officially on maven central.

@greenrobot

This comment has been minimized.

Copy link
Owner

greenrobot commented Nov 15, 2013

Commit 86d14d2 fixes it. (Preferred isEmpty()). Thanks anyway!

@greenrobot greenrobot closed this Nov 15, 2013

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