[mapguide-internals] Rfc 21 posted

[Bruce Dechant]

Here are my comments.

In the APIs below there is no need for the "mapName" parameter because
everything is already contained in the underlying map resource
referenced by the MgResourceIdentifier so the parameter is redundant:

void MgMap::UpdateRuntimeMap(MgResourceService* resourceService,
MgResourceIdentifier* mapDefinition, CREFSTRING mapName); void
MgMap::UpdateMapDefinition(MgResourceService* resourceService,
MgResourceIdentifier* mapDefinition, CREFSTRING mapName);

For the other proposed APIs below the "mapName" parameter should be
replaced with the MgResourceIdentifier because we do operations with
resource IDs:

void MgMap::UpdateRuntimeMap(MgResourceService* resourceService,
MgByteReader* mapDefinitionAsXml, CREFSTRING mapName);
MgByteReader* MgMap::MapDefinitionFromRuntimeMap(MgResourceService*
resourceService, CREFSTRING mapName); void
MgMap::CreateMap(MgResourceService* resourceService, MgByteReader*
mapDefinitionAsXml, CREFSTRING mapName);

Thanks,
Bruce

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org
[mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of
Kenneth, GEOGRAF A/S
Sent: May 29, 2007 2:56 AM
To: MapGuide Internals Mail List
Subject: [mapguide-internals] Rfc 21 posted

I have posted a Rfc.

http://trac.osgeo.org/mapguide/wiki/MapGuideRfc21

-- 
Regards, Kenneth, GEOGRAF A/S


_______________________________________________
mapguide-internals mailing list
mapguide-internals at lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/mapguide-internals