[mapguide-internals] RE: MapGuide RFC 82 is posted for review

[Trevor Wekel]

Hi Klain,

The RFC looks ok but the Doxygen syntax contained in the comments needs to be updated to reflect the correct function signatures.  You may also want to specifically mention all of the derived classes which will implement the new signatures (MgProxyFeatureReader, MgProxyDataReader, MgServerDataReader, MgServerFeatureReader, etc).

Thanks,
Trevor

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Klain Qin
Sent: August 13, 2009 10:28 PM
To: MapGuide Internals Mail List
Subject: [mapguide-internals] MapGuide RFC 82 is posted for review

Hi All,

Please review MapGuide RFC 82 - Extend Reader Access By Index. Any comments are much appreciated. 

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

Thanks,
Klain

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Leaf Li
Sent: Friday, August 14, 2009 9:41 AM
To: mapguide-internals at lists.osgeo.org
Subject: [mapguide-internals] MapGuide RFC 80 is updated.

All,

I update MapGuide RFC 80. Can you review it?

The original solution is to add two new overloaded method ExecuteSqlQuery and ExecuteNoSqlQuery to support parameter binding. The current plan is to extend method ExecuteSqlQuery and ExecuteNoSqlQuery added in MapGuide RFC 78 because those two methods isn't released yet.

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