<br><br><div class="gmail_quote">2011/5/4 Kohei Yoshida <span dir="ltr"><<a href="mailto:kyoshida@novell.com">kyoshida@novell.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <div class="im">On Wed, 2011-05-04 at 18:23 +0200, Markus Mohrhard wrote:<br> > I'm fine with nearly everything. And your suggestion looks even a bit<br> > better than mine, except for the setByTable method.<br> ><br> > I was thinking about adding the table index too, but stumbled over the<br> > case when the table index does not match the table the db data refers<br> > to.<br> <br> </div>Ok. I still wonder in what situations the table index differs from the<br> table of the db data, but I'll take your word for it.<br></blockquote><div><br>At the moment there is no case where this happens, but there may be someone in the future who will think of this as a good idea to solve a problem. It's the same reason made me write the lines that check for the index of the table: as long as something is possible someone will try it. <br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <div class="im"><br> > We have a similar problem in our code base, but as long as we are<br> > aware of it there are no problems. But with the new uno api there<br> > might be an add-on developer who doesn't know that the range and the<br> > table index must match.<br> > So I would prefer to get the information which table is used by the db<br> > range itself. It will still be possible to check for the index to be<br> > out of bounds.<br> <br> </div>Ok. I'm fine with that. Let's go with your suggestion. <br></blockquote><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <font color="#888888"><br> Kohei<br> <br> </font></blockquote></div><br>