openoffice-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jürgen Schmidt <>
Subject Re: [Call-for-Review] code changes for more powerful smarttag extensions
Date Fri, 15 Mar 2013 12:06:27 GMT
On 3/15/13 11:34 AM, Ariel Constenla-Haile wrote:
> On Fri, Mar 15, 2013 at 10:45:56AM +0100, Jürgen Schmidt wrote:
>> On 3/15/13 10:20 AM, Ariel Constenla-Haile wrote:
>>> On Wed, Mar 13, 2013 at 02:30:26PM +0100, Jürgen Schmidt
>>> wrote:
>>>> - a missing include of XInterface in the new IDL 
>>>> XMarkingAccess.idl, IDL compile error on Mac, surprising that
>>>> it worked for you
>>> This is a bug, the one that removed the need for explicitly 
>>> inheriting from XInterface should have taken care for not
>>> needing to include the IDL, what sounds like a non-sense (do
>>> not explicitly inherit, but include the header!).
>> I agree that it's a bug
>>>> The interface name "XMarkingAccess" and the method name 
>>>> "invalidateMarkings" sounds somewhat strange but I have to 
>>>> confess that I don't have a much better name in place. Maybe 
>>>> somebody else has a good name in mind?
>>> IMHO what it does is more problematic than how it's named; see
>>> my comment on the bug.
>> issue
> Not this one, but
> invalidation
> should be triggered on a TextMarkupType base, just like in 
> XFlatParagraph::setChecked, otherwise a smart tag extension
> triggers unnecessary spell and grammar checking.

ok, I see thanks for the pointer. It's probably worth to take a
further look on this code.

@Kai, Ariel has a better overview than I have already. I looked not
deep enough in the code. Please take his feedback into account. We
should avoid any unnecessary overhead.


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message