All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philippe Gerum <rpm@xenomai.org>
To: Dmitry Adamushko <dmitry.adamushko@domain.hid>
Cc: xenomai@xenomai.org
Subject: Re: [Xenomai-core] [Fwd: Re: [Xenomai-help] timeout in native API calls (cond, sem, mutex, etc).]
Date: Sat, 22 Oct 2005 17:22:53 +0200	[thread overview]
Message-ID: <435A594D.2060608@domain.hid> (raw)
In-Reply-To: <200510221551.25114.dmitry.adamushko@domain.hid>

Dmitry Adamushko wrote:
> On Friday 21 October 2005 19:35, Philippe Gerum wrote:
> 
>>Actually, there is a more general problem with the current coding style
>>used throughout the code base: it's mine, it's not that standard,
> 
> 
> then what's standard in this case?
> 

By standard, I mean the one which is the most widely used in the context of 
kernel development, which is relevant to the larger and most active part of the 
Xenomai codebase.

> 
>>and now 
>>thatmore people are contributing to it, I'm pondering whether we should
>>just adoptthe conventional kernel coding style, without the ludicrous
>>8-space tabs, that is.
> 
> 
> The "kernel coding style" is just yet another codding style. If more people 
> are contributing to the project, then it should be a standard that satisfies 
> the most part of them. Is it necessarily the linux way?
> As I told you once, the important thing is how easily the code may be read, 
> hence, understood. IMHO, how/where the braces, etc. are placed is not the 
> matter of the first instance.
> The use of any codding style doesn't result in neat and well-readable code 
> automagically.
> 
> Nevertheless, if the ideas are as follows:
> 
> 1) linux codding style as a part of the seamless integration with the linux 
> kernel. err.., what about a possibility to be merged with the mainline 
> kernel? ok, sounds almost impossible :) ;
> 

Maybe on April 1st.

> 2) linux people are reading/will read the code, there are a lot of 
> style-adherents amongst them, so let's keep them satisfied and our mail boxes 
> free of the messages like "first change your codding style before submitting 
> anything" (btw, I remember one of the answers after publishing of first 
> i-pipe patch was of that kind). btw, there are some parts of the kernel (e.g. 
> some filesystems, if I'm not wrong) that use another style;
> 

Allowing people to agree on a common coding style, which is basically K&R for 
the presentation part, and a few recommendations which belong to common sense, 
is usually the best way to avoid further ludicrous flamewars and unsmart 
assaults from smart asses about the cosmological importance of brace placement 
in the dynamic balance of things in the universe...


> 3) ok, I may assume, that if a person is familiar with some codding style, 
> he/she gets a few percent speed-up when reading the code that is of the same 
> style. At least, for the first few minutes :) But here we must assume that 
> the most part of the potential readers/contributors like the linux codding 
> way.
>

As you pointed out, this is also about allowing people used to K&R style to read 
this code as they would do with any contributed kernel code without any useless 
stylistic barrier on entry, would they be interested in doing so, that is. 
Reading some comments on the LKML about real-time extensions (by opposition to 
native real-time support a la PREEMPT_RT), it seems that there is a fair amount 
of confusion, misinformation and urban legends still going there. Part of the 
reasons for this is very likely that very few actually ever tried to look at the 
code. There must be some reason for that, aside of "they don't care", that is.

> err.. to sum it up, I like the current codding style, but if 1-3) + smth else 
> are worthy things, then why not. I'll get used to another style, if the code 
> will remain err. well-readable :)
> 

The basic idea I had for RTAI/fusion has not changed with Xenomai "reloaded": we 
are working hard to have the highest possible level of integration with the 
Linux kernel while keeping the advantages the co-scheduling approach still has 
over any purely native implementation. In order to achieve this, it would make 
sense to share as many (reasonable) conventions as possible with the kernel people.

> 
>>--
> 
> 
>>Philippe.
>>
> 
> 
> ---
> Best regards,
> Dmitry
> 


-- 

Philippe.


  reply	other threads:[~2005-10-22 15:22 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-21 17:35 [Xenomai-core] [Fwd: Re: [Xenomai-help] timeout in native API calls (cond, sem, mutex, etc).] Philippe Gerum
2005-10-21 17:59 ` Heikki Lindholm
2005-10-22 13:51 ` Dmitry Adamushko
2005-10-22 15:22   ` Philippe Gerum [this message]
2005-10-22 16:49     ` Jan Kiszka
2005-10-22 19:48       ` Gilles Chanteperdrix

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=435A594D.2060608@domain.hid \
    --to=rpm@xenomai.org \
    --cc=dmitry.adamushko@domain.hid \
    --cc=xenomai@xenomai.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.