Skip to content

Fix issue #865 #900

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 1, 2014
Merged

Fix issue #865 #900

merged 1 commit into from
Apr 1, 2014

Conversation

@jgustie
Copy link
Contributor

@jgustie jgustie commented Mar 29, 2014

I figured getInboundNext() could have just been used instead of getFirstInboundFilter() but I wan't sure if the synchronized access in handle would be desirable and opted to add a check to see if there is an inboundNext.

@jlouvel jlouvel added this to the 2.3 M2 milestone Apr 1, 2014
@jlouvel
Copy link
Collaborator

jlouvel commented Apr 1, 2014

Thanks for the pull request. Looks great!
Set milestone to 2.3 M2 and assigned to @thboileau

@thboileau
Copy link
Contributor

thanks a lot Jeremy!

thboileau added a commit that referenced this pull request Apr 1, 2014
@thboileau
Merge pull request #900 from jgustie/compositehelper
0a34c80 
Fix issue #865
@thboileau thboileau merged commit 0a34c80 into restlet:master Apr 1, 2014
@jgustie jgustie deleted the compositehelper branch April 1, 2014 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@thboileau thboileau

Labels

Module: Restlet API Priority: medium State: implementation Type: enhancement

Projects

None yet

Milestone

2.3 M2

Development

Successfully merging this pull request may close these issues.

3 participants

@jgustie @jlouvel @thboileau