[Mapguide-trac] [mapguide-trac] #2524: Review places where argument / null reference exceptions are thrown (was: Review places where argument exceptions are thrown)

[MapGuide Open Source]

#2524: Review places where argument / null reference exceptions are thrown
-----------------------+----------------------------------------------------
   Reporter:  jng      |       Owner:  jng
       Type:  task     |      Status:  new
   Priority:  low      |   Milestone:  3.0
  Component:  General  |     Version:     
   Severity:  trivial  |    Keywords:     
External_id:           |  
-----------------------+----------------------------------------------------
Description changed by jng:

Old description:

> There are still some places in the MapGuide codebase where naked
> MgNullArgumentException objects are thrown.
>
> Cases where these are thrown provide no useful context for the developer
> as it lack key important information: The name of the argument that is
> null.
>
> We already have a CHECKARGUMENTNULL() macro to inject the argument name
> when throwing the exception. We need to find all cases where such
> exceptions are still manually thrown and replace them with this macro.

New description:

 There are still some places in the MapGuide codebase where naked
 MgNullArgumentException and MgNullReferenceException objects are thrown.

 Cases where these are thrown provide no useful context for the developer
 as it lack key important information: The name of the argument that is
 null.

 We already have a CHECKARGUMENTNULL() and CHECKNULL() macros to inject the
 argument name when throwing the exception. We need to find all cases where
 such exceptions are still manually thrown and replace them with this
 macro.

--

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