[Mono-devel-list] [PATCH] implementation ofSystem.Data.DataSet.Merge(...)

Eran Domb erand at mainsoft.com
Mon Sep 29 02:41:28 EDT 2003


Hello Duncan,

Sorry for the compile error.
Here is a new diff file with fix for the compilation problem.

Thanks
Eran.

-----Original Message-----
From: Duncan Mak [mailto:duncan at ximian.com] 
Sent: Thursday, September 25, 2003 8:34 PM
To: eran
Cc: mono-devel mailing list
Subject: Re: [Mono-devel-list] [PATCH] implementation
ofSystem.Data.DataSet.Merge(...)

Hello Eran,

On Mon, 2003-09-22 at 15:04, eran wrote:
> 
> I implemented System.Data.DataSet.Merge(...) and
> System.Data.DataSet.GetChanges(...) and
> System.Data.DataSet.HasChanges(...).
> 
> It also involves changes in System.Data.DataRow.
> 

This patch hasn't been applied to CVS yet because it does not compile
properly:

System.Data/MergeManager.cs(122) error CS0103: The name `Res.GetString'
could not be found in `System.Data.MergeManager'
System.Data/MergeManager.cs(162) error CS0103: The name `Res.GetString'
could not be found in `System.Data.MergeManager'

The variable 'Res' doesn't exist in the file, but we do have a Locale
class, and you can use 'Locale.GetText' to a similar effect.

In addition, I have applied 7 patches you sent to the list on Monday.
They are:

1) missing validation in System.Data.DataRowCollection.Add(DataRow row)
2) bug in System.Data.UniqueConstraint.AssertConstraint(DataRow row)
3) bug in System.Data.Common.DbDataAdapter.Fill(DataSet dataset,
DataTable srcTable, IDataReader dataReader, int startRecord, int
maxRecords)
4) bug in System.Data.DataTableCollection.Add(DataTable table).
5) implementation of System.Data.DataTableCollection.CanRemove(DataTable
table)
6) bug in System.Data.DataRelation.Add(...)
7) implementation of System.Data.DataColumn.Unique

As for all the attached test cases, I really don't have time to look
into them and rewrite them as NUnit TestCases now. In the future, it
would be much more preferable to submit NUnit test cases than simple
discrete programs as a test case, as that helps us improve our existing
testsuite.

Thanks for the contribution!

Duncan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: merge.diff
Type: application/octet-stream
Size: 13906 bytes
Desc: not available
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20030929/a9cb1f88/attachment.obj 


More information about the Mono-devel-list mailing list