RE: CVS update: subversion/subversion/clients/win32/svn_com SVNSt atus.rgs SVNStatus.h SVNStatus.cpp SVNCOMps.mk SVNCOMps.def SVNCOM.rc SVN COM.idl SVNCOM.def SVNCOM.cpp svn_comCP.h svn_com.dsp SVN.rgs SVN.h SVN.c pp StdAfx.h StdAfx.cpp resource.h misc.h mis

> From: Branko Cibej [mailto:brane@xbc.nu]

> rassilon@tigris.org wrote:

> > #ifndef __SVNSTATUS_H_
> > #define __SVNSTATUS_H_
> > #pragma once

> Isn't this a bit of overkill? I mean, either the #ifdef or the #pragma
> is fine, but why use both? (FWIW, I'd use only the #ifdef.)

Just being paranoid. #pragma once didn't always work once upon a time.

> Another thing ... and this will start the old flame war again, yeah ...
> could we please decide to at least not use names that start with
> underscores -- specifically, those that match /^((__)|(_[A-Z]))/ --
> because they are reserved in both C and C++?

> Yes, yes, I know they "can't possibly" clash with anything provided by
> the compiler and standard libraries ...

Yeah, I'll fix that.

Bill
Received on Sat Oct 21 14:36:19 2006