[haiku-development] Re: coding style

  • From: Marcus Overhagen <marcusoverhagen@xxxxxxxx>
  • To: haiku-development@xxxxxxxxxxxxx
  • Date: Mon, 3 Mar 2008 16:08:22 +0100 (CET)

Axel Dörfler  <axeld@xxxxxxxxxxxxxxxx> write:

> "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx> wrote:
> > > another thing:
> > > I would like to have the opening curly bracket of a class or struct
> > > definition
> > > on the next line, as we do when defining functions:
> > >
> > > class Foobar
> > > {
> > You're kidding, right?
> 
> Now I remember you actually do that all the time, so you're probably
> not kidding this time. Anyway, a class is not a function, it's more
> like a struct. It also doesn't make sense to me to change something in
> the style guide that we could easily live with for the past 7 years,
> and that most of our codebase conform to.
> 

I'm not anal about changing it. But I like the form descibed above.

I today noticed that the current coding guidelines do it different 
to what I have done so far.

I thought that most original BeOS header files did it that way and 
put the bracket on a new line, too. I even found some examples:

http://www.beatjapan.org/mirror/www.be.com/developers/developer_library/headers/be/media/AudioStream.h
http://www.beatjapan.org/mirror/www.be.com/developers/developer_library/headers/be/media/Subscriber.h
http://www.beatjapan.org/mirror/www.be.com/developers/developer_library/headers/be/midi/MidiSynth.h
http://www.beatjapan.org/mirror/www.be.com/developers/developer_library/headers/be/midi/MidiSynthFile.h

but there are also examples for our current style:
http://www.beatjapan.org/mirror/www.be.com/developers/developer_library/headers/be/media/VideoModule.h

regards
Marcus


Jetzt komfortabel bei Arcor-Digital TV einsteigen: Mehr Happy Ends, mehr 
Herzschmerz, mehr Fernsehen! Erleben Sie 50 digitale TV Programme und optional 
60 Pay TV Sender, einen elektronischen Programmführer mit Movie Star 
Bewertungen von TV Movie. Außerdem, aktuelle Filmhits und prickelnde Erotik in 
der Arcor-Videothek. Infos unter www.arcor.de/tv

Other related posts: