[GRASS5] (Text based) startup screen patch

GRASS GIS GRASS GIS developer
1

Hi,

IMHO the line length of location/mapset in the text based startup
screen is too short.

I suggest attached patch to 5.3 (linked into 5.7):
src/general/init/set_data.diff

Any problems (it is still shorter than the variable definition)?

Markus

(attachments)

set_data.diff (592 Bytes)

2

Markus Neteler wrote:

Hi,

IMHO the line length of location/mapset in the text based startup
screen is too short.

I suggest attached patch to 5.3 (linked into 5.7):
src/general/init/set_data.diff

Actually, I've had the appended patch lying around for a while; it's
essentially the same thing, only more so.

Any problems (it is still shorter than the variable definition)?

I don't think so.

--
Glynn Clements <glynn.clements@virgin.net>

Index: src/general/init/set_data.c

RCS file: /grassrepository/grass/src/general/init/set_data.c,v
retrieving revision 1.5
diff -u -r1.5 set_data.c
--- src/general/init/set_data.c 13 Mar 2003 08:22:10 -0000 1.5
+++ src/general/init/set_data.c 10 Aug 2004 10:12:45 -0000
@@ -53,7 +53,7 @@
int main (int argc, char *argv)
{
     char version[80];
- char gisdbase[51] ;
+ char gisdbase[70];
     char location_name[41] ;
     char location[1024] ;
     char mapset[41] ;
@@ -102,14 +102,14 @@

   line++;
   V_line (line, loc_text);
- V_ques (location_name, 's', line++, 12, 14);
+ V_ques (location_name, 's', line++, 10, 28);

   V_line (line, map_text);
- V_ques (mapset, 's', line++, 12, 14);
+ V_ques (mapset, 's', line++, 10, 28);

   line++;
   V_line (line, "DATABASE:");
- V_ques (gisdbase, 's', line++, 12, sizeof(gisdbase) - 1);
+ V_ques (gisdbase, 's', line++, 10, sizeof(gisdbase) - 1);

   V_intrpt_ok();
   if (!V_call())

3

On Tue, Aug 10, 2004 at 11:13:13AM +0100, Glynn Clements wrote:

Markus Neteler wrote:

> Hi,
>
> IMHO the line length of location/mapset in the text based startup
> screen is too short.
>
> I suggest attached patch to 5.3 (linked into 5.7):
> src/general/init/set_data.diff

Actually, I've had the appended patch lying around for a while; it's
essentially the same thing, only more so.

> Any problems (it is still shorter than the variable definition)?

I don't think so.

Ok, submitted to CVS.

Markus

4

This should be amply long for complex location/mapset names.

Of course, as it is, I have trouble remembering and correctly typing names
with the current 14 character limit. :wink:

Michael
____________________
C. Michael Barton, Professor
School of Human Origins, Cultures, & Societies
PO Box 872402
Arizona State University
Tempe, AZ 85287-2402
USA

Phone: 480-965-6262
Fax: 480-965-7671
www: <www.public.asu.edu/~cmbarton>

From: Markus Neteler <neteler@itc.it>
Date: Tue, 10 Aug 2004 11:50:16 +0200
To: grass5 developers list <grass5@grass.itc.it>
Subject: [GRASS5] (Text based) startup screen patch

Hi,

IMHO the line length of location/mapset in the text based startup
screen is too short.

I suggest attached patch to 5.3 (linked into 5.7):
src/general/init/set_data.diff

Any problems (it is still shorter than the variable definition)?

Markus