Development Specs: Difference between revisions

From GRASS-Wiki
Jump to navigation Jump to search
(moved to trac)
 
(110 intermediate revisions by 5 users not shown)
Line 1: Line 1:
== Message standardization ==
{{MovedToTrac|MessageStandardization}}
 
=== How should Errors/Warnings/Messages be formatted ===
* strings '''< >'''
** e.g. Raster map <%s> not found
* numbers '''[ ]'''
** e.g. Line [%d] deleted
 
'''Discussion:'''
 
Statistics [2007-04-11]:
<div style="margin-left: 30px">
{|border="1"
!symbol || align="right"| number of code lines
|-
|<%s> || align="right"| 637
|-
|[%s] || align="right"| 690
|-
|'%s' || align="right"| 354
|-
|<%d> || align="right"| 12
|-
|[%d] || align="right"| 207
|-
|'%d' || align="right"| 3
|-
|}
</div>
 
=== Macros to be defined for C library ===
* MSG_RASTER_NOT_FOUND_IN_MAPSET - "Raster map <%s> not found in <%s>"
* MSG_CANNOT_OPEN_RASTER - "Failed opening raster map <%s>"
* MSG_CANNOT_OPEN_FILE - Cannot open file <%s>
 
Note: Problem with xgettext package. How to use macros to work with xgettext?
 
=== Standard messages sandbox ===
 
;DB
:Cannot open database <%s> by driver <%s>
:Cannot insert new record: %s
:Cannot open database <%s> by driver <%s>
:Cannot create table: %s
:Cannot grant privileges on table <%s>
 
;Raster
:Raster map <%s> not found
:Raster map <%s> not found in the current mapset
:Raster map <%s> not found in <%s>
:Cannot read header of raster map <%s>
:Cannot open raster map <%s>
 
;Vector
:Vector map <%s> not found
:Vector map <%s> not found in the current mapset
:Vector map <%s> not found in <%s>
:Cannot open vector map <%s>
:Cannot open vector map <%s> at topo level [%d]
:Cannot get layer info for vector map <%s>
:Cannot create new vector map <%s>
 
''Discussion'':
* "Map <roads> in <user1>" or "Map <roads@user1>"
 
== Systematic Approach ==
* Collect all possible error states from functions
* assign error codes for macros?
* G_option() parameters (like output->description = ("Path to resulting ASCII file");)
 
Note: Would be possible to conctretize your ideas? (MartinL)
 
=== Parameters and flags ===
 
Fix the parameters and flags. Make it a concept. See proposal in GRASS 5 [http://freegis.org/cgi-bin/viewcvs.cgi/grass/documents/parameter_proposal.txt?rev=HEAD&content-type=text/vnd.viewcvs-markup documents/parameter_proposal.txt]
 
== Verbosity levels ==
 
=== Current abstract concept ===
 
{|
|-
|verbose || 3 || MAX_LEVEL || print progress and all messages messages
|-
|standard || 2 || STD_LEVEL || print progress and selected messages
|-
|brief || 1 || || print only progress information
|-
|silent but not mute || 0 || MIN_LEVEL || print nothing (but ERR and WAR)
|-
|mute || 2>&1 > /dev/null
|-
|}
 
;Proposal
 
* module output (should be &quot;parsable&quot;) is not controlled by GRASS_VERBOSE [use fprintf (stdout, ...)]
* silent
** only warnings and fatal errors are printed
* brief
** all G_percent() and selected G_message() (especially connected to progress information, up to the programmer ??)
* standard
** all G_percent() and selected G_message()
* verbose
** all G_percent() + G_message()
 
;Modification
 
* '''remove''' verbose level &rarr; standard == all G_percent () + G_messages()
* move some messages to G_debug()
* change level numbers to 1, 3, 5
* Add new G_msg(int level, "%format", ...); lib function. Existing G_message() could stay and would be the same as G_msg(LEVEL_STD,...).
 
;Discussion
 
* HB: I'm quite happy to leave the current system as it is, but use GRASS_MESSAGE_FORMAT=silent to switch off G_percent(),G_clicker() output.
:Mixing a sliding verbosity scale plus a binary switch or >= rule for when G_percent() happens in the same variable is very messy IMO.
 
=== Alternative Abstract concept ===
 
{|
|-
|standard || 3 || PERMSG_MODE || print progress and messages
|-
|standard || 2 || MESSAGE_MODE || print only messages
|-
|brief || 1 || PERCENT_MODE || print only progress information
|-
|silent but not mute || 0 || QUIET_MODE || print nothing (but ERR and WAR)
|-
|mute || 2>&1 > /dev/null
|-
|}
 
[[Category:Development]]

Latest revision as of 12:42, 2 August 2009