[GRASS-SVN] r49166 - in grass/trunk: lib/python raster/r.info

[svn_grass at osgeo.org]

Author: lucadelu
Date: 2011-11-10 01:46:21 -0800 (Thu, 10 Nov 2011)
New Revision: 49166

Modified:
   grass/trunk/lib/python/raster.py
   grass/trunk/raster/r.info/main.c
Log:
standardize r.info with -g flag for shell output and remove several other flags, like v.info

Modified: grass/trunk/lib/python/raster.py
===================================================================
--- grass/trunk/lib/python/raster.py	2011-11-10 08:33:22 UTC (rev 49165)
+++ grass/trunk/lib/python/raster.py	2011-11-10 09:46:21 UTC (rev 49166)
@@ -73,7 +73,7 @@
         else:
             return float(s)
 
-    s = read_command('r.info', flags = 'rgstmpud', map = map)
+    s = read_command('r.info', flags = 'g', map = map)
     kv = parse_key_val(s)
     for k in ['min', 'max']:
 	kv[k] = float_or_null(kv[k])

Modified: grass/trunk/raster/r.info/main.c
===================================================================
--- grass/trunk/raster/r.info/main.c	2011-11-10 08:33:22 UTC (rev 49165)
+++ grass/trunk/raster/r.info/main.c	2011-11-10 09:46:21 UTC (rev 49166)
@@ -60,9 +60,8 @@
     struct Reclass reclass;
     struct GModule *module;
     struct Option *opt1;
-    struct Flag *rflag, *sflag, *tflag, *gflag, *hflag, *mflag;
-    struct Flag *uflag, *dflag, *timestampflag;
-
+    struct Flag *gflag, *hflag;
+    
     /* Initialize GIS Engine */
     G_gisinit(argv[0]);
 
@@ -74,44 +73,14 @@
 
     opt1 = G_define_standard_option(G_OPT_R_MAP);
 
-    rflag = G_define_flag();
-    rflag->key = 'r';
-    rflag->description = _("Print range only");
-
-    sflag = G_define_flag();
-    sflag->key = 's';
-    sflag->description =
-	_("Print raster map resolution (NS-res, EW-res) only");
-
-    tflag = G_define_flag();
-    tflag->key = 't';
-    tflag->description = _("Print raster map type only");
-
     gflag = G_define_flag();
     gflag->key = 'g';
-    gflag->description = _("Print map region only");
+    gflag->description = _("Print basic info in shell script style");
 
     hflag = G_define_flag();
     hflag->key = 'h';
     hflag->description = _("Print raster history instead of info");
 
-    uflag = G_define_flag();
-    uflag->key = 'u';
-    uflag->description = _("Print raster map data units only");
-
-    dflag = G_define_flag();
-    dflag->key = 'd';
-    dflag->description = _("Print raster map vertical datum only");
-
-    mflag = G_define_flag();
-    mflag->key = 'm';
-    mflag->description = _("Print map title only");
-
-    timestampflag = G_define_flag();
-    timestampflag->key = 'p';
-    timestampflag->description =
-	_("Print raster map timestamp (day.month.year hour:minute:seconds) only");
-
     if (G_parser(argc, argv))
 	exit(EXIT_FAILURE);
 
@@ -145,9 +114,8 @@
 
     out = stdout;
 
-    if (!rflag->answer && !sflag->answer && !tflag->answer &&
-	!gflag->answer && !hflag->answer && !timestampflag->answer &&
-	!mflag->answer && !uflag->answer && !dflag->answer) {
+    /* no flags */
+    if (!gflag->answer && !hflag->answer) {   
 	divider('+');
 
 	compose_line(out, "Layer:    %-29.29s  Date: %s", name,
@@ -320,9 +288,10 @@
 
 	fprintf(out, "\n");
     }
-    else {	/* r,s,t,g,h, or m flag */
+    else {	/* g,h flag */
 
-	if (rflag->answer) {
+	if (gflag->answer) {
+	    /* old rflag */
 	    if (data_type == CELL_TYPE) {
 		if (2 == Rast_read_range(name, "", &crange)) {
 		    fprintf(out, "min=NULL\n");
@@ -341,10 +310,7 @@
 		fprintf(out, "min=%.15g\n", zmin);
 		fprintf(out, "max=%.15g\n", zmax);
 	    }
-
-	}
-
-	if (gflag->answer) {
+	    
 	    G_format_northing(cellhd.north, tmp1, -1);
 	    G_format_northing(cellhd.south, tmp2, -1);
 	    fprintf(out, "north=%s\n", tmp1);
@@ -354,47 +320,46 @@
 	    G_format_easting(cellhd.west, tmp2, -1);
 	    fprintf(out, "east=%s\n", tmp1);
 	    fprintf(out, "west=%s\n", tmp2);
-	}
-
-	if (sflag->answer) {
+	    
+	    /* old sflag */
 	    G_format_resolution(cellhd.ns_res, tmp3, cellhd.proj);
 	    fprintf(out, "nsres=%s\n", tmp3);
 
 	    G_format_resolution(cellhd.ew_res, tmp3, cellhd.proj);
 	    fprintf(out, "ewres=%s\n", tmp3);
-	}
-
-	if (tflag->answer) {
+	    
+	    /* new rows and columns parameters */
+	    fprintf(out, "rows=%d\n", cellhd.rows);
+	    fprintf(out, "cols=%d\n", cellhd.cols);
+	    
+	    /* old tflag */
 	    fprintf(out, "datatype=%s\n",
 		    (data_type == CELL_TYPE ? "CELL" :
 		     (data_type == DCELL_TYPE ? "DCELL" :
 		      (data_type == FCELL_TYPE ? "FCELL" : "??"))));
-	}
-
-	if (mflag->answer) {
+	    
+	    /* old mflag */   
 	    fprintf(out, "title=%s (%s)\n", cats_ok ? cats.title :
 		    "??", hist_ok ? Rast_get_history(&hist, HIST_TITLE) : "??");
-	}
-
-	if (timestampflag->answer) {
+	    
+	    /* old timestampflag */
 	    if (time_ok && (first_time_ok || second_time_ok)) {
 
 		G_format_timestamp(&ts, timebuff);
 
 		/*Create the r.info timestamp string */
 		fprintf(out, "timestamp=\"%s\"\n", timebuff);
-
 	    }
 	    else {
 		fprintf(out, "timestamp=\"none\"\n");
-	    }
+	    }	
+	    
+	    /* old uflag */
+	    fprintf(out, "units=%s\n", units ? units : "(none)");
+	    /* old dflag */
+	    fprintf(out, "vertical_datum=%s\n", vdatum ? vdatum : "(none)");	    
 	}
 
-	if (uflag->answer)
-	    fprintf(out, "units=%s\n", units ? units : "(none)");
-	if (dflag->answer)
-	    fprintf(out, "vertical_datum=%s\n", vdatum ? vdatum : "(none)");
-
 	if (hflag->answer) {
 	    if (hist_ok) {
 		fprintf(out, "Data Source:\n");
@@ -409,8 +374,7 @@
 		}
 	    }
 	}
-    }				/* else rflag or sflag or tflag or gflag or hflag or mflag */
-
+    }
     return EXIT_SUCCESS;
 }