[mapguide][MG234][New] Fix MgStringCollection class

[Walt Welton-Lair]

You can view the artifact detail at the following URL:

    https://mapguide.osgeo.org/servlets/Scarab/id/MG234

Type
 Defect

Artifact ID
 MG234 (Fix MgStringCollection class)

Reported by
 Walt Welton-Lair
 waltweltonlair (walt.welton-lair at autodesk.com)

New artifact details:
---------------------------------------------------------

- Subcomponent set to new value
  Core
- Summary set to new value
  Fix MgStringCollection class
- Component set to new value
  All
- Artifact created
- Version set to new value
  1.0.0
- Target milestone set to new value
  1.0.1
- Priority set to new value
  P2
- Platform set to new value
  All
- Defect Severity set to new value
  High
- Steps to Reproduce set to new value
  The following code throws an exception:

MgStringCollection coll;
coll.Add(L"");

- Artifact Status set to new value
  New
- Description set to new value
  The MgStringCollection class is nothing more than a collection of strings.  Yet it has some weird implementation quirks:

1) You can't add an empty string to the collection - throws an InvalidArgumentException

2) Under the covers it uses an MgPropertyCollection, and each call to MgStringCollection::Add adds an MgStringProperty object to the MgPropertyCollection.  This has the following impact:
- slower performance due to object creation
- more memory usage
- greater code complexity
- causes the problem in 1)

- Operating system set to new value
  All

---------------------------------------------------------
This message was automatically generated by Project Tracker.