[GRASS-SVN] r31506 - in grass/branches/develbranch_6: include lib/bitmap lib/ogsf

svn_grass at osgeo.org svn_grass at osgeo.org
Sat May 24 04:24:16 EDT 2008


Author: martinl
Date: 2008-05-24 04:24:16 -0400 (Sat, 24 May 2008)
New Revision: 31506

Modified:
   grass/branches/develbranch_6/include/gsurf.h
   grass/branches/develbranch_6/lib/bitmap/bitmap.c
   grass/branches/develbranch_6/lib/ogsf/GS2.c
   grass/branches/develbranch_6/lib/ogsf/Gs3.c
   grass/branches/develbranch_6/lib/ogsf/gs.c
   grass/branches/develbranch_6/lib/ogsf/gsds.c
Log:
ogsflib: doxygen strings/debug messages updated


Modified: grass/branches/develbranch_6/include/gsurf.h
===================================================================
--- grass/branches/develbranch_6/include/gsurf.h	2008-05-24 05:44:09 UTC (rev 31505)
+++ grass/branches/develbranch_6/include/gsurf.h	2008-05-24 08:24:16 UTC (rev 31506)
@@ -1,12 +1,20 @@
-/*
-* $Id$
+/*!
+  \file gsurf.h
+ 
+  \brief OGSF library - main header
+ 
+  GRASS OpenGL gsurf OGSF Library 
+ 
+  (C) 1999-2008 by the GRASS Development Team
+ 
+  This program is free software under the 
+  GNU General Public License (>=v2). 
+  Read the file COPYING that comes with GRASS
+  for details.
+  
+  \author Bill Brown USACERL (January 1993)
 */
 
-/*  gsurf.h
-    Bill Brown, USACERL
-    January 1993
-*/
-
 #ifndef _GSURF_H
 #define _GSURF_H
 
@@ -69,17 +77,17 @@
 #define LEGAL_SRC(s) (s==NOTSET_ATT||s==MAP_ATT||s==CONST_ATT||s==FUNC_ATT)
 
 /* site markers */
-#define ST_X        1
+#define ST_X          1
 #define ST_BOX        2
-#define ST_SPHERE    3
+#define ST_SPHERE     3
 #define ST_CUBE       4
-#define ST_DIAMOND     5
-#define ST_DEC_TREE     6
-#define ST_CON_TREE     7
-#define ST_ASTER    8
-#define ST_GYRO            9
+#define ST_DIAMOND    5
+#define ST_DEC_TREE   6
+#define ST_CON_TREE   7
+#define ST_ASTER      8
+#define ST_GYRO       9
 /* ACS_MODIFY_One Line site_attr management ***************************************/
-#define ST_HISTOGRAM       10
+#define ST_HISTOGRAM  10
 
 /* site attribute modes (what attribute of marker category is used for) */
 /* TODO: may want to make these OR'able */

Modified: grass/branches/develbranch_6/lib/bitmap/bitmap.c
===================================================================
--- grass/branches/develbranch_6/lib/bitmap/bitmap.c	2008-05-24 05:44:09 UTC (rev 31505)
+++ grass/branches/develbranch_6/lib/bitmap/bitmap.c	2008-05-24 08:24:16 UTC (rev 31506)
@@ -47,11 +47,14 @@
 
 /*!
  * \brief Create bitmap of dimension x/y and return structure token. 
+ *
  * Bitmap is initialized to all zeros
  *
- *  \param int x
- *  \param int y
- *  \return struct BM or NULL on error
+ *  \param x x dimension
+ *  \param y y dimension
+ *
+ *  \return pointer to struct BM
+ *  \return NULL on error
  */
 
 struct BM *BM_create (int x, int y)

Modified: grass/branches/develbranch_6/lib/ogsf/GS2.c
===================================================================
--- grass/branches/develbranch_6/lib/ogsf/GS2.c	2008-05-24 05:44:09 UTC (rev 31505)
+++ grass/branches/develbranch_6/lib/ogsf/GS2.c	2008-05-24 08:24:16 UTC (rev 31506)
@@ -83,10 +83,11 @@
 }
 
 /*!
-  \brief Library initialization
+  \brief Initialize OGSF library
 
-  Get region settings - wind, Region (NSWE array)
-  Compute scale
+  Get region settings - wind
+
+  Set Region (NSWE array) and compute scale
 */
 void GS_libinit(void)
 {
@@ -213,8 +214,6 @@
 {
     geosurf *ns;
 
-    G_debug(3, "GS_new_surface");
-
     if (Next_surf < MAX_SURFS) {
 	ns = gs_get_new_surface();
 	gs_init_surf(ns, wind.west + wind.ew_res / 2.,
@@ -228,9 +227,13 @@
 	Surf_ID[Next_surf] = ns->gsurf_id;
 	++Next_surf;
 
+	G_debug(3, "GS_new_surface(): id=%d", ns->gsurf_id);
+	
 	return (ns->gsurf_id);
     }
 
+
+
     return (-1);
 }
 
@@ -1569,11 +1572,11 @@
 
 
 /*!
-  \brief Load attribute map
+  \brief Load raster map as attribute
 
   \param id surface id
   \param filename filename
-  \param att attribute id
+  \param att attribute descriptor
 
   \return -1 on error (invalid surface id)
   \return ?
@@ -1588,7 +1591,7 @@
     int reuse = 0, begin, hdata, ret, neg = 0, has_null = 0;
     typbuff *tbuff;
 
-    G_debug (3, "GS_load_att_map(): att_map: %s", filename);
+    G_debug (3, "GS_load_att_map(): map=%s", filename);
 
     gs = gs_get_surf(id);
 
@@ -1747,8 +1750,6 @@
 				      tbuff->nm, &has_null);
 	    filename = G_fully_qualified_name(filename, mapset);
 
-	    G_debug(3, "HAS-NULL = %d", has_null);
-
 	    break;
 	case ATTY_INT:
 	default:
@@ -1769,7 +1770,7 @@
 	    return (-1);
 	}
 
-	G_debug(3, "HAS-NULL = %d", has_null);
+	G_debug(4, "  has_null=%d", has_null);
 
 	if (!has_null) {
 	    gsds_free_data_buff(gs->att[att].hdata, ATTY_NULL);
@@ -1843,9 +1844,6 @@
 	G_warning(_("Error finding range"));
     }
 
-    G_debug (3, "Range Updated: %f %f",
-	     gs->zmin, gs->zmax);
-
     return (ret);
 }
 
@@ -2068,7 +2066,7 @@
   \brief Set draw mode
 
   \param id surface id
-  \param mode mode id
+  \param mode mode type(s)
 
   \return 0 on success
   \return -1 on error (invalid surface id)
@@ -2077,7 +2075,8 @@
 {
     geosurf *gs;
 
-    G_debug(3, "GS_set_drawmode");
+    G_debug(3, "GS_set_drawmode(): id=%d mode=%d",
+	    id, mode);
 
     gs = gs_get_surf(id);
 
@@ -2215,7 +2214,8 @@
 {
     geosurf *gs;
 
-    G_debug(3, "GS_set_drawres");
+    G_debug(3, "GS_set_drawres() id=%d xyres=%d/%d xywire=%d/%d",
+	    id, xres, yres, xwire, ywire);
 
     if (xres < 1 || yres < 1 || xwire < 1 || ywire < 1) {
 	return (-1);
@@ -2272,8 +2272,6 @@
 {
     geosurf *gs;
 
-    G_debug(3, "GS_get_dims");
-
     gs = gs_get_surf(id);
 
     if (gs) {

Modified: grass/branches/develbranch_6/lib/ogsf/Gs3.c
===================================================================
--- grass/branches/develbranch_6/lib/ogsf/Gs3.c	2008-05-24 05:44:09 UTC (rev 31505)
+++ grass/branches/develbranch_6/lib/ogsf/Gs3.c	2008-05-24 08:24:16 UTC (rev 31506)
@@ -130,7 +130,7 @@
     char *map_set, *nullflags;
     int offset, row, col;
 
-    G_debug(3, "Gs_loadmap_as_float");
+    G_debug(3, "Gs_loadmap_as_float(): name=%s", map_name);
 
     map_set = G_find_cell2(map_name, "");
     if (!map_set) {
@@ -167,7 +167,7 @@
 	}
     }
 
-    G_debug(3, "_HAS-NULL_ = %d", *has_null);
+    G_debug(4, "  has_null=%d", *has_null);
 
     G_close_cell(cellfile);
 
@@ -1132,10 +1132,10 @@
 }
 
 /*!
-  \brief Update no_zero ranges for att (actually no_null now)
+  \brief Update no_zero ranges for attribute (actually no_null now)
   
   \param gs pointer to geosurf struct
-  \param desc attribute id
+  \param desc attribute id (descriptor)
 
   \return -1 on error
   \return 1 on success
@@ -1246,5 +1246,8 @@
 	gs->zrange_nz = gs->zmax_nz - gs->zmin_nz;
     }
 
+    G_debug (3, "Gs_update_attrange(): min=%f max=%f",
+	     gs->zmin, gs->zmax);
+
     return (1);
 }

Modified: grass/branches/develbranch_6/lib/ogsf/gs.c
===================================================================
--- grass/branches/develbranch_6/lib/ogsf/gs.c	2008-05-24 05:44:09 UTC (rev 31505)
+++ grass/branches/develbranch_6/lib/ogsf/gs.c	2008-05-24 08:24:16 UTC (rev 31506)
@@ -39,7 +39,7 @@
 }
 
 /*!
-  \brief Library initialization
+  \brief Initialize library
 
   Still need to take care of library initialization, 
   probably want to define a Surf_top of constant value (i.e., 0)
@@ -63,10 +63,11 @@
 {
     geosurf *gs;
 
-    G_debug(3, "gs_get_surf");
+    G_debug(4, "gs_get_surf():");
 
     for (gs = Surf_top; gs; gs = gs->next) {
 	if (gs->gsurf_id == id) {
+	    G_debug(4, "  id=%d", id);
 	    return (gs);
 	}
     }
@@ -86,7 +87,7 @@
 {
     geosurf *ps;
 
-    G_debug(3, "gs_get_prev_surface");
+    G_debug(4, "gs_get_prev_surface");
 
     for (ps = Surf_top; ps; ps = ps->next) {
 	if (ps->gsurf_id == id - 1) {
@@ -113,7 +114,7 @@
 	gsurfs[i] = gs;
     }
 
-    G_debug(3, "gs_num_surfaces(): num=%d", i);
+    G_debug(4, "gs_num_surfaces(): num=%d", i);
 
     return (i);
 }
@@ -130,7 +131,7 @@
 
     for (i = 0, gs = Surf_top; gs; gs = gs->next, i++);
 
-    G_debug(3, "gs_num_surfaces(): num=%d", i);
+    G_debug(4, "gs_num_surfaces(): num=%d", i);
 
     return (i);
 }
@@ -164,7 +165,7 @@
 }
 
 /*!
-  \brief Get last geosurf struct
+  \brief Get last allocated geosurf struct from list
 
   \return pointer to geosurf struct
 */
@@ -172,22 +173,21 @@
 {
     geosurf *ls;
 
-    G_debug(3, "gs_get_last_surface():");
-
     if (!Surf_top) {
 	return (NULL);
     }
 
     for (ls = Surf_top; ls->next; ls = ls->next);
 
-    G_debug(3, " last surface id: %d", ls->gsurf_id);
+    G_debug(4, "gs_get_last_surface(): last surface id=%d",
+	    ls->gsurf_id);
 
     return (ls);
 }
 
 
 /*!
-  \brief Get new geosurf struct
+  \brief Allocate new geosurf struct
 
   \return pointer to geosurf struct
 */
@@ -195,8 +195,6 @@
 {
     geosurf *ns, *ls;
 
-    G_debug(3, "gs_get_new_surface");
-
     ns = (geosurf *) G_malloc(sizeof(geosurf)); /* G_fatal_error */
     if (!ns) {
 	return (NULL);
@@ -213,13 +211,15 @@
 
     ns->next = NULL;
 
+    G_debug(4, "gs_get_new_surface(): id=%d", ns->gsurf_id);
+
     return (ns);
 }
 
 /*!
-  \brief Init geosurf struct
+  \brief Initialize allocated geosurf struct
 
-  Now xmin & ox are the same, right? - get rid of ox, oy in geosurf struct?
+  \todo Now xmin & ox are the same, right? - get rid of ox, oy in geosurf struct?
 
   \param gs pointer to geosurf struct
   \param ox,oy x/y origin coordinates
@@ -236,7 +236,7 @@
     geosurf *ps;
     int i;
 
-    G_debug(3, "gs_init_surf");
+    G_debug(4, "gs_init_surf() id=%d", gs->gsurf_id);
 
     if (!gs) {
 	return (-1);
@@ -422,17 +422,17 @@
 }
 
 /*!
-  \brief Set default attributes
+  \brief Set default attribute values
 
   \param gs pointer to geosurf struct
-  \param defs array of default values
-  \param null_defs array of null default values
+  \param defs array of default values (dim MAX_ATTRS)
+  \param null_defs array of null default values (dim MAX_ATTRS)
 */
 void gs_set_defaults(geosurf * gs, float *defs, float *null_defs)
 {
     int i;
 
-    G_debug(3, "gs_set_defaults");
+    G_debug(4, "gs_set_defaults(): id=%d", gs->gsurf_id);
 
     for (i = 0; i < MAX_ATTS; i++) {
 	gs->att[i].constant = defs[i];
@@ -454,7 +454,7 @@
 {
     geosurf *fs;
 
-    G_debug(3, "gs_delete_surf");
+    G_debug(4, "gs_delete_surf");
     
     fs = gs_get_surf(id);
 
@@ -479,7 +479,7 @@
     geosurf *gs;
     int found = 0;
 
-    G_debug(3, "gs_free_surf");
+    G_debug(4, "gs_free_surf");
 
     if (Surf_top) {
 	if (fs == Surf_top) {
@@ -550,7 +550,7 @@
     int i, j, same;
     int old_datah;
 
-    G_debug(3, "gs_free_unshared_buffs");
+    G_debug(4, "gs_free_unshared_buffs");
 
     /* for each attribute 
        if !same, free buff   
@@ -589,7 +589,7 @@
     geosurf *gs;
     int ref, j;
 
-    G_debug(3, "gs_num_datah_reused");
+    G_debug(4, "gs_num_datah_reused");
 
     /* for each attribute 
        if same, ++reference
@@ -619,7 +619,7 @@
 */
 int gs_get_att_type(geosurf * gs, int desc)
 {
-    G_debug(3, "gs_get_att_type");
+    G_debug(4, "gs_get_att_type");
 
     if (!LEGAL_ATT(desc)) {
 	return (-1);
@@ -645,7 +645,7 @@
 */
 int gs_get_att_src(geosurf * gs, int desc)
 {
-    G_debug(3, "gs_get_att_src");
+    G_debug(4, "gs_get_att_src(): id=%d desc=%d", gs->gsurf_id, desc);
 
     if (!LEGAL_ATT(desc)) {
 	return (-1);
@@ -662,8 +662,8 @@
   \brief Get attribute typbuff
 
   \param gs pointer to geosurf struct
-  \param desc attribute id
-  \param to_write
+  \param desc attribute id (descriptor)
+  \param to_write non-zero value for 'write'
 
   \return NULL on error
   \return pointer to typbuff
@@ -673,6 +673,9 @@
     typbuff *tb;
     geosurf *gsref;
 
+    G_debug(4, "gs_get_att_typbuff(): id=%d desc=%d to_write=%d",
+	    gs->gsurf_id, desc, to_write);
+
     if (gs) {
 	if ((tb = gsds_get_typbuff(gs->att[desc].hdata, to_write))) {
 	    tb->tfunc = NULL;
@@ -696,8 +699,8 @@
   \brief Allocate attribute buffer
 
   \param gs pointer to geosurf struct
-  \param desc attribute id
-  \param type buffer type
+  \param desc attribute id (descriptor)
+  \param type buffer type (based on raster map type)
 
   \return -1 on error
   \return pointer to typbuff (casted)
@@ -706,7 +709,7 @@
 {
     int hdata, dims[2], ndims;
 
-    G_debug(3, "gs_malloc_att_buff");
+    G_debug(4, "gs_malloc_att_buff");
 
     if (gs) {
 	if (0 < (hdata = gs->att[desc].hdata)) {
@@ -735,7 +738,7 @@
 {
     int size;
 
-    G_debug(3, "gs_malloc_lookup");
+    G_debug(4, "gs_malloc_lookup");
 
     if (gs) {
 	if (gs->att[desc].lookup) {
@@ -791,7 +794,7 @@
 int gs_set_att_type(geosurf * gs, int desc, int type)
 {
 
-    G_debug(3, "gs_set_att_type(): desc=%d, type=%d",
+    G_debug(4, "gs_set_att_type(): desc=%d, type=%d",
 	    desc, type);
 
     if (gs && LEGAL_TYPE(type)) {
@@ -807,7 +810,7 @@
   \brief Set attribute source
 
   \param gs pointer to geosurf struct
-  \param desc attribute id
+  \param desc attribute id (descriptor)
   \param src source id
 
   \return -1 on error
@@ -815,7 +818,8 @@
 */
 int gs_set_att_src(geosurf * gs, int desc, int src)
 {
-    G_debug(3, "gs_set_att_src");
+    G_debug(4, "gs_set_att_src(): id=%d desc=%d src=%d",
+	    gs->gsurf_id, desc, src);
 
     /* check if old source was MAP_ATT, free buff */
     if (MAP_ATT == gs_get_att_src(gs, desc)) {
@@ -858,7 +862,7 @@
 */
 int gs_set_att_const(geosurf * gs, int desc, float constant)
 {
-    G_debug(3, "gs_set_att_const");
+    G_debug(4, "gs_set_att_const");
 
     if (gs) {
 	gs->att[desc].constant = constant;

Modified: grass/branches/develbranch_6/lib/ogsf/gsds.c
===================================================================
--- grass/branches/develbranch_6/lib/ogsf/gsds.c	2008-05-24 05:44:09 UTC (rev 31505)
+++ grass/branches/develbranch_6/lib/ogsf/gsds.c	2008-05-24 08:24:16 UTC (rev 31506)
@@ -1,7 +1,7 @@
 /*!
   \file gsds.c
  
-  \brief OGSF library - ? (lower level functions) 
+  \brief OGSF library - dataset loading and management (lower level functions) 
  
   GRASS OpenGL gsurf OGSF Library 
 
@@ -79,7 +79,7 @@
 static int Tot_mem = 0;
 
 /*!
-  \brief Initialize 
+  \brief Initialize gsds
 */
 static int init_gsds(void)
 {
@@ -211,7 +211,7 @@
 /*!
   \brief Get handle to gsds
 
-  \param name
+  \param name raster map name
 
   \return -1 on failure
   \return data id
@@ -266,16 +266,14 @@
 }
 
 /*!
-  \brief Get typbuff
+  \brief Get data buffer
 
-  change_flag just tells us to set changed flag
+  Doesn't prevent writing a buff thats's been gotten with change_flag
+  == 0 (could return a copy, but willing to trust calling func for
+  now)
   
-  Doesn't prevent
-  writing a buff thats's been gotten with change_flag == 0 (could return a
-  copy, but willing to trust calling func for now)
-  
-  \param id
-  \param change_flag
+  \param id dataset id
+  \param change_flag set changed flag
 
   \return pointer to typbuff struct
   \return NULL on failure
@@ -359,10 +357,10 @@
 }
 
 /*!
-  \brief ADD
+  \brief Free allocated buffer
 
-  \param id
-  \param typ
+  \param id dataset id
+  \param typ data type
 
   \return 0 not found
   \return 1 found
@@ -384,12 +382,12 @@
 }
 
 /*!
-  \brief ADD
+  \brief Free data buffer
 
-  \param ds
-  \param typ
+  \param ds pointer to dataset struct
+  \param typ data type
 
-  \return
+  \return freed size
 */
 int free_data_buffs(dataset * ds, int typ)
 {
@@ -457,8 +455,8 @@
     ds->numbytes -= freed;
 
     if (freed) {
-	G_debug (3, "freed data from id no. %d", ds->data_id);
-	G_debug (3, "%.3f Kbytes freed, current total = %.3f",
+	G_debug (5, "free_data_buffs(): freed data from id no. %d", ds->data_id);
+	G_debug (5, "free_data_buffs(): %.3f Kbytes freed, current total = %.3f",
 		 freed / 1000., Tot_mem / 1000.);
     }
     
@@ -470,10 +468,10 @@
 
   \todo add ATTY_CONST
 
-  \param id
-  \param dims
-  \param ndims
-  \param type
+  \param id dataset id
+  \param dims array of dimensions
+  \param ndims number of dimensions
+  \param type data type
 
   \return
 */
@@ -590,7 +588,7 @@
 	ds->ndims = ndims;
 	Tot_mem += siz;
 
-	G_debug (3, "%f Kbytes allocated, current total = %f",
+	G_debug (5, "gsds_alloc_typbuff(): %f Kbytes allocated, current total = %f",
 		 siz / 1000., Tot_mem / 1000.);
 
 	return (siz);



More information about the grass-commit mailing list