[brlcad-commits] SF.net SVN: brlcad:[38267] brlcad/trunk/src/libged
Open Source Solid Modeling CAD
Brought to you by:
brlcad
From: <br...@us...> - 2010-03-31 05:34:42
|
Revision: 38267 http://brlcad.svn.sourceforge.net/brlcad/?rev=38267&view=rev Author: brlcad Date: 2010-03-31 05:34:36 +0000 (Wed, 31 Mar 2010) Log Message: ----------- quell minor warnings, add extra param checking Modified Paths: -------------- brlcad/trunk/src/libged/draw.c brlcad/trunk/src/libged/dup.c Modified: brlcad/trunk/src/libged/draw.c =================================================================== --- brlcad/trunk/src/libged/draw.c 2010-03-31 05:02:08 UTC (rev 38266) +++ brlcad/trunk/src/libged/draw.c 2010-03-31 05:34:36 UTC (rev 38267) @@ -192,8 +192,12 @@ static union tree * -ged_wireframe_region_end(struct db_tree_state *tsp, const struct db_full_path *pathp, union tree *curtree, genptr_t client_data) +ged_wireframe_region_end(struct db_tree_state *tsp, const struct db_full_path *pathp, union tree *curtree, genptr_t client_data __attribute__((unused))) { + if (tsp) RT_CK_DBTS(tsp); + if (pathp) RT_CK_FULL_PATH(pathp); + if (curtree) RT_CK_TREE(curtree); + return (curtree); } @@ -837,11 +841,15 @@ static union tree * -ged_bot_check_region_end(struct db_tree_state *tsp, +ged_bot_check_region_end(struct db_tree_state *tsp, const struct db_full_path *pathp, - union tree *curtree, - genptr_t client_data) + union tree *curtree, + genptr_t client_data __attribute__((unused))) { + if (tsp) RT_CK_DBTS(tsp); + if (pathp) RT_CK_FULL_PATH(pathp); + if (curtree) RT_CK_TREE(curtree); + return curtree; } @@ -1152,7 +1160,7 @@ continue; } - if (strlen( argv[i] ) == (1 + (ptr_A != NULL) + (ptr_o != NULL))) { + if (strlen( argv[i] ) == ((size_t)1 + (ptr_A != NULL) + (ptr_o != NULL))) { /* argv[i] is just a "-A" or "-o" */ continue; } Modified: brlcad/trunk/src/libged/dup.c =================================================================== --- brlcad/trunk/src/libged/dup.c 2010-03-31 05:02:08 UTC (rev 38266) +++ brlcad/trunk/src/libged/dup.c 2010-03-31 05:34:36 UTC (rev 38267) @@ -172,7 +172,8 @@ name = (char *)rip->name.ext_buf; - if ( name == (char *)NULL ) return; + if (name == (char *)NULL) return; + if (addr == 0) return; /* do not compare _GLOBAL */ if ( rip->major_type == DB5_MAJORTYPE_ATTRIBUTE_ONLY && @@ -216,7 +217,7 @@ * Check a name against the global directory. */ static int -ged_dir_check(struct db_i *input_dbip, const char *name, size_t laddr, size_t len, int flags, genptr_t ptr) +ged_dir_check(struct db_i *input_dbip, const char *name, size_t laddr __attribute__((unused)), size_t len __attribute__((unused)), int flags __attribute__((unused)), genptr_t ptr) { struct directory *dupdp; struct bu_vls local; This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |