nikhilgupta10 / GridLAB-D

Other
1 stars 0 forks source link

#555 --example command option outputs debug dump instead of GLM code, #2300

Closed nikhilgupta10 closed 8 years ago

nikhilgupta10 commented 8 years ago

The new command line option added in changeset:2970 outputs a debugging dump of the object instead of a GLM save of the object. The call in cmdarg.c/example() needs to be changed from object_dump() to object_save(), a new function to be split out from object_saveall() that copy the object data into a buffer.

,

nikhilgupta10 commented 8 years ago

nikhilgupta10 imported these comments from Sourceforge: The user dchassin does not exist anymore. Therefore assigning this to afisher1. "dchassin": * status changed from new to accepted ,

"dchassin":object.c/objectsave(char,sizet,OBJECT) is implemented in changeset:2972 but only linked into cmdarg.c/example(int,char*[]). Still todo item to link it into object.c/object_saveall() because the existing object output implementation how defects that need to reviewed to consider the potential impact of changes to object_saveall() before being resolved. In particular:

  1. object_saveall() does not put single quotes around timestamps. It probably should.

  2. object_saveall() does not put double quotes around empty strings and non-numeric values that have white spaces. It probably should.

There may be other issues relating to other GLD data types that should be reviewed too before making the final linkage between object_save() and object_saveall().

,

"dchassin":Documentation is updated at https://sourceforge.net/apps/mediawiki/gridlab-d/index.php?title=Example.

,

"dchassin": * status changed from accepted to assigned ,

"dchassin": * status changed from assigned to closed

The command works well enough for the purpose of getting a list of available properties and their defaults.

,