[Mapguide-trac] [mapguide-trac] #2013: Improve error message quality of exceptions

[MapGuide Open Source]

#2013: Improve error message quality of exceptions
---------------------------+------------------------------------------------
   Reporter:  jng          |       Owner:     
       Type:  enhancement  |      Status:  new
   Priority:  low          |   Milestone:  2.4
  Component:  General      |     Version:     
   Severity:  trivial      |    Keywords:     
External_id:               |  
---------------------------+------------------------------------------------
Description changed by jng:

Old description:

> There are a series of exceptions thrown in MapGuide that could be easily
> improved in terms of clarity. For example:
>
>  * MgNullArgumentException - Should include the identifier that failed
> the null pointer check
>  * MgNullReferenceException - Should include the identifier that failed
> the null reference check
>  * MgObjectNotFoundException - Should include the name whose associated
> object could not be found
>
> Most of these exceptions are thrown by pre-defined macros like
> CHECKARGUMENTNULL(), etc. So a simple modification to these macros where
> applicable to include the stringified argument being checked will result
> in clearer error messages being included.

New description:

 There are a series of exceptions thrown in MapGuide that could be easily
 improved in terms of clarity. For example:

  * MgNullArgumentException - Should include the identifier that failed the
 null pointer check
  * MgNullReferenceException - Should include the identifier that failed
 the null reference check
  * MgArgumentOutOfRangeException - Should include the identifier that
 failed the range check
  * MgObjectNotFoundException - Should include the name whose associated
 object could not be found

 Most of these exceptions are thrown by pre-defined macros like
 CHECKARGUMENTNULL(), etc. So a simple modification to these macros where
 applicable to include the stringified argument being checked will result
 in clearer error messages being included.

--

-- 
Ticket URL: <http://trac.osgeo.org/mapguide/ticket/2013#comment:2>
MapGuide Open Source <http://mapguide.osgeo.org/>
MapGuide Open Source Internals