-
Notifications
You must be signed in to change notification settings - Fork 216
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
APNS resend queue #39
Conversation
{stop, normal, State}; | ||
|
||
handle_cast({in, Msg}, #state{max_entries=MaxEntries,queue=OldQueue}=State) -> | ||
Queue = case MaxEntries =< ?QUEUE:len(OldQueue) of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We would prefer avoiding using macros for function or module names.
Fixed: altenwald/apns4erl@b59e7a0 |
@manuel-rubio somehow that commit did not get into the PR |
Added into pull request |
|
||
handle_cast({in, Msg}, #state{max_entries=MaxEntries,queue=OldQueue}=State) -> | ||
Queue = case MaxEntries =< queue:len(OldQueue) of | ||
true -> queue:liat(OldQueue); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just as a comment, I would avoid the Okasaki version of the queue api.
Looks good, some tests would be welcome :) |
Fix for issue #17
Commit is adopted for version 1.0.2 from two commits:
altenwald@06431ef
altenwald@618e52c