[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 <anagappan at novell.com>
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 <informatique.internet at fiducial.fr> 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.