----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65169/#review195425 -----------------------------------------------------------
src/server.c Line 580 (original), 580 (patched) <https://reviews.apache.org/r/65169/#comment274607> On a second look, the qd_sever_t *sever parameter to set_rhost_port might not be necessary. It is not used inside the function. - Ganesh Murthy On Jan. 15, 2018, 3:53 p.m., Gordon Sim wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65169/ > ----------------------------------------------------------- > > (Updated Jan. 15, 2018, 3:53 p.m.) > > > Review request for qpid, Chug Rolke, Ganesh Murthy, and Ted Ross. > > > Bugs: DISPATCH-912 > https://issues.apache.org/jira/browse/DISPATCH-912 > > > Repository: qpid-dispatch > > > Description > ------- > > Done now first thing on remote-open (rather than in connection-bound as > before). > > > Diffs > ----- > > src/server.c 5cf967e > > > Diff: https://reviews.apache.org/r/65169/diff/2/ > > > Testing > ------- > > Tests pass for me with this patch. > > > Thanks, > > Gordon Sim > >
