cvs commit: fptools/ghc/includes Block.h Bytecodes.h ClosureMacros.h ClosureTypes.h Closures.h Cmm.h Constants.h GranSim.h Hooks.h HsFFI.h InfoTables.h Linker.h MachRegs.h Makefile Parallel.h README Regs.h RtsExternal.h RtsFlags.h ...

[Simon Marlow]

On 27 March 2005 14:41, Sven Panne wrote:

> panne       2005/03/27 05:41:19 PST
> 
>   Modified files:
>     ghc/includes         Block.h Bytecodes.h ClosureMacros.h
>                          ClosureTypes.h Closures.h Cmm.h
>                          Constants.h GranSim.h Hooks.h HsFFI.h
>                          InfoTables.h Linker.h MachRegs.h Makefile
>                          Parallel.h README Regs.h RtsExternal.h
>                          RtsFlags.h RtsMessages.h RtsTypes.h SMP.h
>                          STM.h SchedAPI.h Signals.h Stg.h StgFun.h
>                          StgProf.h StgTicky.h Storage.h TSO.h
>                          TailCalls.h Updates.h
>                          mkDerivedConstants.c
>     ghc/rts              Apply.h Arena.h AutoApply.h Capability.h
>                          GC.c HeapStackCheck.cmm Interpreter.h
>                          Linker.c MBlock.h ProfHeap.c Profiling.c
>                          RetainerProfile.c RetainerSet.c
>                          RetainerSet.h RtsFlags.c RtsUtils.h
>                          Sanity.c Stats.h
>     ghc/rts/parallel     Dist.h
>     libraries/base/cbits runProcess.c
>     libraries/base/include HsBase.h
>   Log:
>   * Some preprocessors don't like the C99/C++ '//' comments after a
>     directive, so use '/* */' instead. For consistency, a lot of '//'
>     in the include files were converted, too.

Actually I would have gone the other way and converted them all to //.
I rather like //, especially since /*...*/ doesn't nest properly.  The
coding style document (in the commentary) also mentions that the use of
// is encouraged.

Not that I think we should change them all back; it's not that big a
deal.

Cheers,
	Simon