--
Adam Shannon
Developer
University of Northern Iowa
Junior -- Computer Science & Mathematics
http://ashannon.us
n8han
unread,
Feb 17, 2013, 10:56:38 AM2/17/13
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to dispatc...@googlegroups.com
Hi Adam, this looks like a good direction to move in! Originally I wanted to stick to the underlying `RequestBuilder` type for simplicity. But it seems like making the underlying type immutable in Java will be harder than wrapping it in Scala and providing two-way conversions, as I think you're doing here. This is also of course how Dispatch Classic does things.
Nathan
Jack Viers
unread,
Mar 1, 2013, 1:50:44 PM3/1/13
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to dispatc...@googlegroups.com
I can concur that immutable RequestBuilders are a nice-to-have. Got tripped up on that a lot when I started using dispatch, especially when testing against unfiltered. I defer to the BDFL, but pending some independent testing and code review by n8han, I'd like to see this get added.
Jack Viers
unread,
Mar 1, 2013, 1:52:38 PM3/1/13
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to dispatc...@googlegroups.com
Also, it would help if there were additions to the docs on this, before it gets added.
On Sunday, February 17, 2013 9:56:38 AM UTC-6, n8han wrote: