Changes between Initial Version and Version 1 of Ticket #2524


Ignore:
Timestamp:
Jan 26, 2015 6:32:42 AM (5 years ago)
Author:
jng
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2524

    • Property Summary changed from Review places where argument exceptions are thrown to Review places where argument / null reference exceptions are thrown
  • Ticket #2524 – Description

    initial v1  
    1 There are still some places in the MapGuide codebase where naked MgNullArgumentException objects are thrown.
     1There are still some places in the MapGuide codebase where naked MgNullArgumentException and MgNullReferenceException objects are thrown.
    22
    33Cases 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.
    44
    5 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.
     5We 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.