comp.lang.ada
 help / color / mirror / Atom feed
From: Jacob Sparre Andersen <jacob@jacob-sparre.dk>
Subject: Re: Optimizing Ada
Date: Wed, 02 Oct 2013 10:11:07 +0200
Date: 2013-10-02T10:11:07+02:00	[thread overview]
Message-ID: <87eh84f5yc.fsf@adaheads.sparre-andersen.dk> (raw)
In-Reply-To: 2cdd0d7d-d62c-42fc-9e5c-beed749f425a@googlegroups.com

kennethesills@gmail.com writes:

> Other than that, is the code itself clean/idiomatic enough? Not
> teaching myself any bad habits?

There are some style issues:

- space before parenthesis (i.e. "procedure Increment_By_One (...").
- lower-case keywords (i.e. "array", "others", ...).

You may get a performance gain by using an extended return statement:

   return Word_Map : Word_Count_Maps.Map do
      ...
   end return;

Greetings,

Jacob
-- 
"Science is like sex: sometimes something useful comes out,
 but that is not the reason we are doing it"
                                          -- Richard Feynman

  reply	other threads:[~2013-10-02  8:11 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-02  2:58 Optimizing Ada kennethesills
2013-10-02  3:47 ` Jeffrey Carter
2013-10-02  3:53   ` kennethesills
2013-10-02  4:13     ` Jeffrey Carter
2013-10-02  4:24       ` kennethesills
2013-10-02  8:11         ` Jacob Sparre Andersen [this message]
2013-10-02 10:32           ` Marius Amado-Alves
2013-10-02 14:24           ` kennethesills
2013-10-02 16:41         ` Jeffrey Carter
2013-10-02 18:58       ` John B. Matthews
2013-10-02  7:01 ` Egil H H
2013-10-02  7:16 ` Simon Wright
2013-10-02 14:43   ` kennethesills
2013-10-02 10:24 ` Marius Amado-Alves
2013-10-02 14:29   ` kennethesills
replies disabled

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox