Jean

I was too quick. Afer refreshing my memory a little bit I'd like to state what follows.

Actually <string.h> in C++ is a C header and totally different from <string>. I guess some buggy compilers treat <string.h> like <string> but that's nonconforming.
I propose to remove all <xxx.h> headers from sources (I'm withdrawing my earlier proposition to change <xxx.h> to <cxxx>).
We could introduce macro INCLUDE(X) and define it eihter to include <xxx> style headers or <xxx.h> style headers for buggy compilers but I wouldn't do that. Every macro takes 5% of users away :)

Regards
Piotr Dobrogost

-------- Original Message --------
Subject: [cURLpp] Compilation fixes from Bob Ham <rah_at_bash.sh> - it shouldn't be like this :)
Date: Tue, 18 Nov 2008 22:23:08 +0100
From: Piotr Dobrogost <curlpp_at_2008.autoera.pl>
Reply-To: cURLpp's mailing-list <curlpp_at_rrette.com>
To: cURLpp's mailing-list <curlpp_at_rrette.com>

Jean

Some of Bob's changes resulted in string header included twice;

curlpp/CurlHandle.cpp:

#include <string>
#include <iostream>
#include <string.h>

We should introduce MACRO informing weather std string (<string>) is present or not and include either <string> or <string.h>
Shall I make changes?

Regards
Piotr Dobrogost
_______________________________________________
cURLpp mailing list
cURLpp_at_rrette.com
http://www.rrette.com/mailman/listinfo/curlpp

_______________________________________________
cURLpp mailing list
cURLpp_at_rrette.com
http://www.rrette.com/mailman/listinfo/curlpp
Received on 2008-11-18