In message <[EMAIL PROTECTED]> on Wed,  5 Nov 2003 08:42:39 +0100 (MET), "Jeffrey 
Altman via RT" <[EMAIL PROTECTED]> said:

rt> 
rt> The inclusion of "e_os.h" in crypto\des\cfb_enc.c must be specified as 
rt> either
rt> 
rt>   #include "openssl/e_os.h"

Absolutely not!

rt> or
rt> 
rt>   #include "../e_os.h"

I'd rather say either:

        #ifdef FLAT_INC
        # include "e_os.h"
        #else
        # include "../../e_os.h"
        #endif

or:

        #include "e_os.h"

I prefer the latter.

rt> This is not performed in a consistent manner in OpenSSL 0.9.6.

Uhmm, why does cbc_enc.c need e_os.h?

But you're right, the consistency is a bit lacking there...

-----
Please consider sponsoring my work on free software.
See http://www.free.lp.se/sponsoring.html for details.

-- 
Richard Levitte   \ Tunnlandsvägen 3  \ [EMAIL PROTECTED]
[EMAIL PROTECTED]  \ S-168 36  BROMMA  \ T: +46-8-26 52 47
                    \      SWEDEN       \ or +46-708-26 53 44
Procurator Odiosus Ex Infernis                -- [EMAIL PROTECTED]
Member of the OpenSSL development team: http://www.openssl.org/

Unsolicited commercial email is subject to an archival fee of $400.
See <http://www.stacken.kth.se/~levitte/mail/> for more info.
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [EMAIL PROTECTED]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to