[Mono-dev] Performance improvments in DataRow

[Hubert FONGARNAND]

Hi,

I've posted a patch that resolve a performance problem on DataRow...
(ADO.NET)

see : http://bugzilla.ximian.com/show_bug.cgi?id=80075

with this patch, my ported ASP.NET is now at 0.93 second for 1 page (it
was 2s before...)



___
Ce message et les éventuels documents joints peuvent contenir des informations 
confidentielles.
Au cas où il ne vous serait pas destiné, nous vous remercions de bien vouloir 
le supprimer et en aviser immédiatement l'expéditeur. Toute utilisation de ce 
message non conforme à sa destination, toute diffusion ou publication, totale 
ou partielle et quel qu'en soit le moyen est formellement interdite.
Les communications sur internet n'étant pas sécurisées, l'intégrité de ce 
message n'est pas assurée et la société émettrice ne peut être tenue pour 
responsable de son contenu.
___
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list

Re: [Mono-dev] Performance improvments in DataRow

[A Nagappan]

Hi,
  Nice work :)

Few comments about the patch (only with respect to coding standards ;).

Add required spacing where ever appropriate.

+   Doublet d= (Doublet)columnNameCount[name.ToLower()];
+   if (d!=null)

something like

+   Doublet d = (Doublet) columnNameCount [name.ToLower ()];
+   if (d != null)

Modify the following

+   if (d!=null)
+   {

as

+   if (d!=null) {
+

Also, add a generic test-case.

Thanks
Nagappan

Nagappan A [EMAIL PROTECTED]
Linux Desktop Testing Project - http://ldtp.freedesktop.org
http://nagappanal.blogspot.com

Novell, Inc.
SUSE® Linux Enterprise 10
Your Linux is ready™
http://www.novell.com/linux


 Hubert FONGARNAND [EMAIL PROTECTED] 11/28/06 10:35 PM 
Hi,

I've posted a patch that resolve a performance problem on DataRow...
(ADO.NET)

see : http://bugzilla.ximian.com/show_bug.cgi?id=80075

with this patch, my ported ASP.NET is now at 0.93 second for 1 page (it
was 2s before...)



___
Ce message et les Θventuels documents joints peuvent contenir des informations 
confidentielles.
Au cas o⋅ il ne vous serait pas destinΘ, nous vous remercions de bien vouloir 
le supprimer et en aviser immΘdiatement l'expΘditeur. Toute utilisation de ce 
message non conforme α sa destination, toute diffusion ou publication, totale 
ou partielle et quel qu'en soit le moyen est formellement interdite.
Les communications sur internet n'Θtant pas sΘcurisΘes, l'intΘgritΘ de ce 
message n'est pas assurΘe et la sociΘtΘ Θmettrice ne peut Ωtre tenue pour 
responsable de son contenu.

___
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list