BIND 10 #168: separate ASIO from Boost and remove custom code

BIND 10 Development do-not-reply at isc.org
Thu May 27 17:10:08 UTC 2010


#168: separate ASIO from Boost and remove custom code
--------------------------+-------------------------------------------------
 Reporter:  larissas      |        Owner:  each                                       
     Type:  task          |       Status:  reviewing                                  
 Priority:  major         |    Milestone:  04. 2nd Incremental Release: Early Adopters
Component:  Unclassified  |   Resolution:                                             
 Keywords:                |    Sensitive:  0                                          
--------------------------+-------------------------------------------------

Comment(by each):

 > Okay, I figured it out.  That's because there are some functions that
 hit the "unused-parameter" warning in header files of the non boost
 version of ASIO.  I suspect we were free from this in the boost version
 because we used a separate object library where the source code that would
 hit the warning is hidden.

 Odd... it compiles fine for me, with the code in the branch.  I guess it's
 because there are boost headers in /usr/include that avoid the unused
 parameter warning somehow?

 > that is, {{{UNUSED_PARAM}}} is specified.  I guess Evan explicitly added
 this to the original source code.

 Ah, yes I had.  It was intended as a temporary hack, which I would go back
 and resolve in some other way after I had the rest of the code working,
 and then I promptly forgot about it. :)

 > I've not come up with a satisfactory solution, but I'm attaching a patch
 that I think is least evil: it introduces a small wrapper module
 (asio_link) as a separate internal "library" (only used for b10-auth), and
 exclude unused-parameter from the erroneous warnings only for that
 library.

 This seems reasonable.  I don't see your new asio_link.{cc,h} files in the
 diff you attached (I guess you did an "svn diff" but forgot the "svn
 add"), but please go ahead and commit the changes to the trac168 branch.
 I'll revert the configure.ac changes, rearrange the "ext" directory, and
 ensure that the local copy of the asio headers are included regardless of
 --with-boost-include.  Thank you for your help on this.

-- 
Ticket URL: <https://bind10.isc.org/ticket/168#comment:13>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list