From mboxrd@z Thu Jan  1 00:00:00 1970
X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on polar.synack.me
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00
	autolearn=unavailable autolearn_force=no version=3.4.4
Path: 
 eternal-september.org!reader01.eternal-september.org!reader02.eternal-september.org!news.eternal-september.org!news.eternal-september.org!.POSTED!not-for-mail
From: Simon Wright <simon@pushface.org>
Newsgroups: comp.lang.ada
Subject: Re: feedback asked on dab-decoder software in Ada
Date: Sun, 25 Sep 2016 10:04:44 +0100
Organization: A noiseless patient Spider
Message-ID: <lyintk5p03.fsf@pushface.org>
References: <52cfa89e-7e4f-48f6-93b4-d559ea001d82@googlegroups.com>
	<87oa3ph71j.fsf@adaheads.sparre-andersen.dk>
	<0dbca7f0-791b-4eee-99fa-ce5fe2a71de6@googlegroups.com>
Mime-Version: 1.0
Content-Type: text/plain
Injection-Info: mx02.eternal-september.org;
 posting-host="e5f31545d127a6738a4229c67d648b1c";
	logging-data="31536"; mail-complaints-to="abuse@eternal-september.org";
	posting-account="U2FsdGVkX1/P4rhXcNQOXjmNFQG718jNeC5O3rhBBB4="
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (darwin)
Cancel-Lock: sha1:PE1io+yA1rqEQmqYyrLWCrZSdyg=
	sha1:Wxn9ibqEF807pzmArC7nsrWaM0c=
Xref: news.eternal-september.org comp.lang.ada:31881
Date: 2016-09-25T10:04:44+01:00
List-Id: <comp.lang.ada>

jan van katwijk <j.vankatwijk@gmail.com> writes:

> Op donderdag 15 september 2016 11:00:10 UTC+2 schreef Jacob Sparre Andersen:
>> Jan van Katwijk wrote:

>> + It would make it easier to get contributions from other Ada
>>   developers, if you switched source style to something closer
>>   to what is suggested by the Ada Quality and Style Guide [1].

> Wrt identifiers: I am not impressed very much by the guidelines, to me
> Is_Running is not better readable than isRunning, Has_Property not
> than hasProperty.

But Jacob is writing from the viewpoint of other Ada developers. I would
personally find using CamelCase very awkward; my IDE takes care of most
Ada-style casing for me, I would have to put isRunning into a special
exception (and the leading 'l' would pretty much mandate that).

>> + Why do you put the package specifications in a separate directory?

I've noticed this a lot recently looking at Ada translations from C
originals. In some cases (e.g. this commit[1]) developers have reverted
to a more 'normal' Ada style, specs & bodies in the same directory and
no '_Pack' on the end of package names!


All this is of course personal preference.

[1] https://github.com/AdaCore/Certyflie/commit/c98fd04ce811b3009de7dd7ab39b0359041b42ff