[mapguide-internals] Patch for ticket 1304: Possible bug in constructor MgSelection (MgMap *map, CREFSTRING xmlSelectionString) triggered by empty xmlSelectionString

[Walt Welton-Lair]

It looks like ClearSelections is also deleting the stream (m_stream).  It should only process / clear m_selections.  Just fix that and then I think the patch looks good.

Walt

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Jackie Ng
Sent: Tuesday, June 12, 2012 3:39 AM
To: mapguide-internals at lists.osgeo.org
Subject: Re: [mapguide-internals] Patch for ticket 1304: Possible bug in constructor MgSelection (MgMap *map, CREFSTRING xmlSelectionString) triggered by empty xmlSelectionString

Hi Walt,

I've put up a 2nd version for review based on your feedback (1304_v2.patch)

- Jackie

--
View this message in context: http://osgeo-org.1560.n6.nabble.com/Patch-for-ticket-1304-Possible-bug-in-constructor-MgSelection-MgMap-map-CREFSTRING-xmlSelectionString-tp4980283p4980663.html
Sent from the MapGuide Internals mailing list archive at Nabble.com.
_______________________________________________
mapguide-internals mailing list
mapguide-internals at lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/mapguide-internals