Re: Ada Shootout program for K-Nucleotide (patches)

[Georg Bauhaus <rm.dash-bauhaus@futureapps.de>]

Martin schrieb:
> On Aug 27, 10:05 am, Martin <martin.do...@btopenworld.com> wrote:
>> Just tried to build the new version from the shootout site and it
>> doesn't build:
>>
>> 46:16     warning: unreachable code
>>
>> in Data_Input.Read_Section.
>>
>> And sure enough I can't see how you get out the outer loop...
>>
>> Cheers
>> -- Martin
> 
> Sorry, it does build - it's a warning. My question is - if you can't
> get out of the loop (except by exception) what are these lines doing
> (other than making the SLOC metric very slightly worse)?


They read data into the package global buffer which is
then used for producing the result of the Read function.  The loop
(and all) is odd indeed but "justified" by the fact that
Skip_To_Section reads until the third (and last) section begins.
Read_Section starts there.  A second "justification" is
that Read_Section is a dumb rewrite of a previously recursive
routine that concatenated strings (thereby hitting stack
limits, no TCE, and making the test fail at the site).

I had seen that warning, too, but other things being
more pressing...
We are in the process of preparing the multitasking version
for submission.  How should we change Read_Section?