CURL review request

Jonathan M Davis jmdavisProg at gmx.com
Tue Aug 16 13:01:15 PDT 2011


On Tuesday, August 16, 2011 18:47:44 Jacob Carlborg wrote:
> On 2011-08-16 15:54, kennytm wrote:
> > Jonas Drewsen<jdrewsen at nospam.com>  wrote:
> >> Hi all,
> >> 
> >>     This is a review request for the curl wrapper. Please read the
> >>     "known>> 
> >> issues" in the top of the source file and if possible suggest a
> >> solution.
> >> 
> >> We also need somebody for running the review process. Anyone?
> >> 
> >> Code:
> >> 	https://github.com/jcd/phobos/blob/curl-wrapper/etc/curl.d
> >> 
> >> Docs:
> >> 	http://freeze.steamwinter.com/D/web/phobos/etc_curl.html
> >> 
> >> Demolish!
> >> 
> >> /Jonas
> > 
> > 1) Why 'Http' and 'Ftp' but 'SMTP' is in all caps?
> > 2) Some links in the doc says 'std.curl....'
> 
> BTW, do we have style guides for abbreviations like this?

No. The style guide says camelcase, but it doesn't specify how to deal with 
abbreviations. Personally, I think that it should be dealt with on a case by 
case basis, since what works best depends on the abbreviation, but in general, 
I prefer all caps.

- Jonathan M Davis


More information about the Digitalmars-d mailing list