Re: [tor-dev] [Patch] or/onion_ntor.c

2013-02-19 Thread Nick Mathewson
On Tue, Feb 12, 2013 at 5:53 PM, Gisle Vanem wrote: > I suggest that the whole body of this file gets included > inside an "#ifdef CURVE25519_ENABLED" in-case someone > (like me!) inadvertently tries to compile it w/o this define > set. Patch: NAK on this patch -- it doesn't make sense IMO to do

[tor-dev] [Patch] or/onion_ntor.c

2013-02-12 Thread Gisle Vanem
I suggest that the whole body of this file gets included inside an "#ifdef CURVE25519_ENABLED" in-case someone (like me!) inadvertently tries to compile it w/o this define set. Patch: --- Git-latest\src\or\onion_ntor.c Tue Feb 12 18:40:04 2013 +++ or\onion_ntor.c Tue Feb 12 23:46:51 2013 @@