Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2003:
[Freeciv-Dev] Re: (PR#6388) Unknown research in intelligence dialog
Home

[Freeciv-Dev] Re: (PR#6388) Unknown research in intelligence dialog

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: remi.bonnet@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#6388) Unknown research in intelligence dialog
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Fri, 3 Oct 2003 08:33:20 -0700
Reply-to: rt@xxxxxxxxxxxxxx

Remi Bonnet wrote:
> Jason Short wrote:
> 
> 
>>Remi Bonnet wrote:
>> 
>>
>>
>>>Another tiny patch ;-)
>>>
>>>This one remove the (0/1) string after 'Researching: (Unknown)' in the
>>>intelligence report when the player don't know the player research.
>>>   
>>>
>>
>>Why do we rarely use spaces in these strings?  E.g., why not have
>>
>>  Republic (10/40)
>>
>>instead of
>>
>>  Republic(10/40)
>>
>>?  The same thing happens in many other strings in the client.  Is there 
>>a reason?
>>
>>jason
>> 
>>
> 
> I don't understand... My patch doesn't deal with spaces...
> 
> It just replace:
>  Researching: (Unknown) (0/1)
> to
>  Researching: (Unknown)
> 
> The first looks stupid for me.

Your patch doesn't introduce this.  But it actually replaces

   Researching: (Unknown)(0/1)

with

   Researching: (Unknown)

so my question is, why don't we use a space in the original string?

jason




[Prev in Thread] Current Thread [Next in Thread]