[Libreoffice] minor idl fixes

Michael Stahl mstahl at redhat.com
Thu Jan 5 04:57:35 PST 2012


On 05/01/12 09:38, Tomas Hlavaty wrote:
> Hi Stephan,
> 
> Stephan Bergmann <sbergman at redhat.com> writes:
> 
>> On 01/04/2012 11:09 PM, Tomas Hlavaty wrote:
>>
>>> diff --git a/offapi/com/sun/star/chart/XAxisSupplier.idl
>>> b/offapi/com/sun/star/chart/XAxisSupplier.idl
>>> ...
>>
>> Are you sure that this is right, using chart2 entities from within
>> chart?  I always intuitively thought that chart2 was a second attempt,
>> to replace chart.
> 
> I am not sure as I don't know what chart and chart2 are about but
> currently this is broken.  The patch fixes the compilation issue.  Maybe
> I should move the file to chart2 package instead?

argh... that would not work, because there is already a chart2::XAxis.
in addition to chart::XAxis.  actually the chart::XAxis is newer than
chart2::XAxis: it was added for OOo 3.4, in commit
0c1fb57fe1588ba0d2ed07145fd07e834c96623f

the implementation of this is in
chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx, and it uses
chart::XAxis.

by now i'm now pretty confused, but i think the proposed patch is wrong.

ah, your trouble could be caused by chart/XAxis.idl and chart2/XAxis.idl
having the same include guard, i'll try to fix that...

grmbl, what a mess...



More information about the LibreOffice mailing list