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=ham
	autolearn_force=no version=3.4.4
X-Google-Thread: 103376,bdcca6db8294fb00
X-Google-NewGroupId: yes
X-Google-Attributes: gida07f3367d7,domainid0,public,usenet
X-Google-Language: ENGLISH,ASCII
Path: 
 g2news1.google.com!news2.google.com!npeer03.iad.highwinds-media.com!news.highwinds-media.com!feed-me.highwinds-media.com!nntp.club.cc.cmu.edu!feeder.erje.net!nuzba.szn.dk!news.szn.dk!pnx.dk!dotsrc.org!filter.dotsrc.org!news.dotsrc.org!not-for-mail
Date: Mon, 20 Dec 2010 10:36:11 +0100
From: =?ISO-8859-1?Q?Thomas_L=F8cke?= <tl@ada-dk.org>
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US;
 rv:1.9.2.13) Gecko/20101210 Thunderbird/3.1.7
MIME-Version: 1.0
Newsgroups: comp.lang.ada
Subject: Re: Introducing memcache-ada, a memcached client in Ada
References: <slrnigt9l7.jge.tyler@kiwi.local>
 <4d0f1316$0$23759$14726298@news.sunsite.dk>
 <4d0f1ed7$0$6990$9b4e6d93@newsspool4.arcor-online.net>
In-Reply-To: <4d0f1ed7$0$6990$9b4e6d93@newsspool4.arcor-online.net>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit
Message-ID: <4d0f238b$0$23754$14726298@news.sunsite.dk>
Organization: SunSITE.dk - Supporting Open source
NNTP-Posting-Host: 83.91.213.86
X-Trace: news.sunsite.dk
 DXC=97QKg[W\R@cCf4eYcI3F5hYSB=nbEKnkk4d83VQloLlo1GQX8;5?CngRED9SjB8:6iQo^8G8>AndmODbDC6l5bAkaBPdSh0TI0d
X-Complaints-To: staff@sunsite.dk
Xref: g2news1.google.com comp.lang.ada:16060
Date: 2010-12-20T10:36:11+01:00
List-Id: <comp.lang.ada>

On 2010-12-20 11:16, Georg Bauhaus wrote:
> On 20/12/2010 09:25, Thomas L�cke wrote:
>> I do though have four suggestions:
>>
>> 1. Declare all the string literals ("STORED", "NOT_FOUND" and such) as
>> constants in the specification.
>
> Not sure why should they be public?  These are protocol words
> that I think a user will not want to deal with.


Do they have to be made public in order to be declared as constants in
the specification? Couldn't they be declared in the private part of the
package?

I'd prefer declaring this once:

Not_Found : constant String := "NOT_FOUND";

And then using Not_Found throughout the package, instead of having
multiple "NOT_FOUND" literals.

OTOH I do see your point about visibility.  :o)

-- 
Regards,
Thomas L�cke

Email: tl at ada-dk.org
Web: http:ada-dk.org
IRC nick: ThomasLocke