[Libreoffice] [Patch] New Uno Interface for sheet local anonymous db data

Kohei Yoshida kyoshida at novell.com
Wed May 4 09:39:22 PDT 2011


On Wed, 2011-05-04 at 18:23 +0200, Markus Mohrhard wrote:
> I'm fine with nearly everything. And your suggestion looks even a bit
> better than mine, except for the setByTable method.
> 
> I was thinking about adding the table index too, but stumbled over the
> case when the table index does not match the table the db data refers
> to. 

Ok.  I still wonder in what situations the table index differs from the
table of the db data, but I'll take your word for it.

> We have a similar problem in our code base, but as long as we are
> aware of it there are no problems. But with the new uno api there
> might be an add-on developer who doesn't know that the range and the
> table index must match.
> So I would prefer to get the information which table is used by the db
> range itself. It will still be possible to check for the index to be
> out of bounds.

Ok.  I'm fine with that.  Let's go with your suggestion.

Kohei



More information about the LibreOffice mailing list