cURL
Haxx ad
libcurl

curl's project page on SourceForge.net

Sponsors:
Haxx

cURL > Mailing List > Monthly Index > Single Mail

curlpp mailing list Archives

Re: [cURLpp] [Fwd: Compilation fixes from Bob Ham <rah@bash.sh> - it shouldn't be like this :)]

From: Jean-Philippe Barette-LaPierre <jpbarrette_at_gmail.com>
Date: Wed, 19 Nov 2008 08:52:08 -0500

Took from dinkumware.com (P. J.
Plauger<http://en.wikipedia.org/wiki/P._J._Plauger>
):

Include the standard header *<cstring>* to define the macros traditionally
defined in the Standard C library header
<string.h><http://www.dinkumware.com/manuals/default.aspx?manual=compleat&page=string.html>.
Including this header also ensures that the names declared with external
linkage in the Standard C library header are declared in the
std<http://www.dinkumware.com/manuals/default.aspx?manual=compleat&page=lib_cpp.html#std%20namespace>namespace.
In this
implementation<http://www.dinkumware.com/manuals/default.aspx?manual=compleat&page=index.html#implementation>,
the names may or may not also be declared in the global namespace, depending
on the specific translation environment.

So we should just change <string.h> to <cstring>

On Tue, Nov 18, 2008 at 5:49 PM, Piotr Dobrogost <curlpp_at_2008.autoera.pl>wrote:

> 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>).

string.h

>
> 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
>

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

These mail archives are generated by hypermail.

donate! Page updated November 12, 2010.
web site info

File upload with ASP.NET