This is the mail archive of the mauve-patches@sourceware.org mailing list for the Mauve project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: FYI: GapContent test fix


Roman Kennke wrote:

Hi David,

Am Donnerstag, den 02.02.2006, 11:16 +0000 schrieb David Gilbert:


Hi Roman,

The API docs describe the first parameter (where) as:

|"where| - the starting position >= 0, < length()"

...and in this particular test I've used length(), so I'd expect the BadLocationException to be thrown. To back this up further, the exact same test on StringContent *does* throw the exception. I didn't hear back from Sun yet on the bug report (apart from the automated response), but I do expect that they'll accept it and fix it eventually.



Now that is interesting. So I was a little too quick with my fix. It seemed to make sense to be able to append, but if StringContent behaves differently, I am not so sure anymore. So you think we should revert my patch?

/Roman



I think so, but if you're busy it's not so important - I'll revisit it when I hear back from Sun on the bug report.

Regards,

Dave


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]