You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by Eike Rathke <oo...@erack.de> on 2011/08/29 02:36:40 UTC
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main:
binfilter/bf_sfx2/source/doc/ binfilter/bf_starmath/source/
binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Hi Michael,
On Sunday, 2011-08-28 23:34:31 -0000, mst@apache.org wrote:
> URL: http://svn.apache.org/viewvc?rev=1162610&view=rev
> Log:
> unotools: OOutputStreamWrapper:
>
> remove invocation of DECLARE_UNO3_AGG_DEFAULTS
> (symbols are somehow not exported with GCC 4.6, and it seems to be
> unnecessary)
The "somehow" may be related to the -fPIC vs -fpic I mentioned in an
earlier mail. Simply removing the DECLARE_UNO3_AGG_DEFAULTS(...) removes
forwarding of acquire() and release() calls to the baseclass, I don't
think that's correct. See comphelper/inc/comphelper/uno3.hxx
Eike
--
PGP/OpenPGP/GnuPG encrypted mail preferred in all private communication.
Key ID: 0x293C05FD - 997A 4C60 CE41 0149 0DB3 9E96 2F1A D073 293C 05FD
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/ binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/ slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by eric b <er...@free.fr>.
Answering to myself, looks like ArchLinux too switched for gcc-4.6.x.
Probably other distributions did.
--
qɔᴉɹə
Projet OOo4Kids : http://wiki.ooo4kids.org/index.php/Main_Page
L'association EducOOo : http://www.educoo.org
Blog : http://eric.bachard.org/news
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/ binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/ slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by eric b <er...@free.fr>.
Hi Mathias,
Le 29 août 11 à 10:44, Mathias Bauer a écrit :
> Which distribution comes with gcc 4.6 as default?
The only one I'm aware is Debian armhf (experimental at the moment),
due to some asm optimisations for Neon and the FPU.
>
Regards,
Eric
--
qɔᴉɹə
Education Project:
http://wiki.services.openoffice.org/wiki/Education_Project
Projet OOo4Kids : http://wiki.ooo4kids.org/index.php/Main_Page
L'association EducOOo : http://www.educoo.org
Blog : http://eric.bachard.org/news
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/
binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Michael Stahl <ms...@openoffice.org>.
On 29.08.2011 10:44, Mathias Bauer wrote:
> On 29.08.2011 10:35, Michael Stahl wrote:
>
>>> Is it wise to fix the code for gcc 4.6 *now*? Every fix might introduce
>>> new bugs and gcc 4.6 is not the "standard" compiler nowadays (AFAIK).
>>
>> please note that i will have a difficult time contributing if i can't
>> build the code :)
>
> Which distribution comes with gcc 4.6 as default?
Fedora 15, for one.
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main:
binfilter/bf_sfx2/source/doc/ binfilter/bf_starmath/source/
binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Eike Rathke <oo...@erack.de>.
Hi Mathias,
On Monday, 2011-08-29 10:44:56 +0200, Mathias Bauer wrote:
> Which distribution comes with gcc 4.6 as default?
Debian wheezy.
Eike
--
PGP/OpenPGP/GnuPG encrypted mail preferred in all private communication.
Key ID: 0x293C05FD - 997A 4C60 CE41 0149 0DB3 9E96 2F1A D073 293C 05FD
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/
binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Mathias Bauer <Ma...@gmx.net>.
On 29.08.2011 10:35, Michael Stahl wrote:
>> Is it wise to fix the code for gcc 4.6 *now*? Every fix might introduce
>> new bugs and gcc 4.6 is not the "standard" compiler nowadays (AFAIK).
>
> please note that i will have a difficult time contributing if i can't
> build the code :)
Which distribution comes with gcc 4.6 as default?
Regards,
Mathias
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/
binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Michael Stahl <ms...@openoffice.org>.
On 29.08.2011 09:36, Mathias Bauer wrote:
> On 29.08.2011 02:36, Eike Rathke wrote:
>> Hi Michael,
>>
>> On Sunday, 2011-08-28 23:34:31 -0000, mst@apache.org wrote:
>>
>>> URL: http://svn.apache.org/viewvc?rev=1162610&view=rev
>>> Log:
>>> unotools: OOutputStreamWrapper:
>>>
>>> remove invocation of DECLARE_UNO3_AGG_DEFAULTS
>>> (symbols are somehow not exported with GCC 4.6, and it seems to be
>>> unnecessary)
>>
>> The "somehow" may be related to the -fPIC vs -fpic I mentioned in an
>> earlier mail.
i vaguely remember having tried this many weeks ago, but it didn't seem
to help. perhaps i tried it on the wrong file...
presumably it should make a difference on x86_64.
>> Simply removing the DECLARE_UNO3_AGG_DEFAULTS(...) removes
>> forwarding of acquire() and release() calls to the baseclass, I don't
>> think that's correct. See comphelper/inc/comphelper/uno3.hxx
but OOutputStreamWrapper is derived from ::cppu::WeakImplHelper1, which
implements the acquire()/release() methods itself (public). so client
code should simply call these base class methods and the manual
forwarding done by the macro seems entirely superfluous to me. (there
are certainly already hundreds of classes that derive from
WeakImplHelper* but don't override acquire()/release().)
> Is it wise to fix the code for gcc 4.6 *now*? Every fix might introduce
> new bugs and gcc 4.6 is not the "standard" compiler nowadays (AFAIK).
please note that i will have a difficult time contributing if i can't
build the code :)
> Regards,
> Mathias
regards,
michael
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/
binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Mathias Bauer <Ma...@gmx.net>.
On 29.08.2011 10:29, Stephan Bergmann wrote:
> On Aug 29, 2011, at 9:36 AM, Mathias Bauer wrote:
>> On 29.08.2011 02:36, Eike Rathke wrote:
>>> Hi Michael,
>>>
>>> On Sunday, 2011-08-28 23:34:31 -0000, mst@apache.org wrote:
>>>
>>>> URL: http://svn.apache.org/viewvc?rev=1162610&view=rev
>>>> Log:
>>>> unotools: OOutputStreamWrapper:
>>>>
>>>> remove invocation of DECLARE_UNO3_AGG_DEFAULTS
>>>> (symbols are somehow not exported with GCC 4.6, and it seems to be
>>>> unnecessary)
>>>
>>> The "somehow" may be related to the -fPIC vs -fpic I mentioned in an
>>> earlier mail. Simply removing the DECLARE_UNO3_AGG_DEFAULTS(...) removes
>>> forwarding of acquire() and release() calls to the baseclass, I don't
>>> think that's correct. See comphelper/inc/comphelper/uno3.hxx
>>
>> Is it wise to fix the code for gcc 4.6 *now*? Every fix might introduce new bugs and gcc 4.6 is not the "standard" compiler nowadays (AFAIK).
>
> I guess the best way to go about it would be to pick an environment where OOo's current OOO340 head is known to build fine and try building AOOo's trunk in that environment. Don't modify too many parameters at once… ;)
More or less that's my point, yes.
Regards,
Mathias
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/ binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/ slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Stephan Bergmann <st...@googlemail.com>.
On Aug 29, 2011, at 9:36 AM, Mathias Bauer wrote:
> On 29.08.2011 02:36, Eike Rathke wrote:
>> Hi Michael,
>>
>> On Sunday, 2011-08-28 23:34:31 -0000, mst@apache.org wrote:
>>
>>> URL: http://svn.apache.org/viewvc?rev=1162610&view=rev
>>> Log:
>>> unotools: OOutputStreamWrapper:
>>>
>>> remove invocation of DECLARE_UNO3_AGG_DEFAULTS
>>> (symbols are somehow not exported with GCC 4.6, and it seems to be
>>> unnecessary)
>>
>> The "somehow" may be related to the -fPIC vs -fpic I mentioned in an
>> earlier mail. Simply removing the DECLARE_UNO3_AGG_DEFAULTS(...) removes
>> forwarding of acquire() and release() calls to the baseclass, I don't
>> think that's correct. See comphelper/inc/comphelper/uno3.hxx
>
> Is it wise to fix the code for gcc 4.6 *now*? Every fix might introduce new bugs and gcc 4.6 is not the "standard" compiler nowadays (AFAIK).
I guess the best way to go about it would be to pick an environment where OOo's current OOO340 head is known to build fine and try building AOOo's trunk in that environment. Don't modify too many parameters at once… ;)
-Stephan
Re: svn commit: r1162610 - in /incubator/ooo/trunk/main: binfilter/bf_sfx2/source/doc/
binfilter/bf_starmath/source/ binfilter/bf_svtools/source/misc/ chart2/source/view/main/
slideshow/source/engine/shapes/ svtools/source/misc/ unotools/inc/unotools/
Posted by Mathias Bauer <Ma...@gmx.net>.
On 29.08.2011 02:36, Eike Rathke wrote:
> Hi Michael,
>
> On Sunday, 2011-08-28 23:34:31 -0000, mst@apache.org wrote:
>
>> URL: http://svn.apache.org/viewvc?rev=1162610&view=rev
>> Log:
>> unotools: OOutputStreamWrapper:
>>
>> remove invocation of DECLARE_UNO3_AGG_DEFAULTS
>> (symbols are somehow not exported with GCC 4.6, and it seems to be
>> unnecessary)
>
> The "somehow" may be related to the -fPIC vs -fpic I mentioned in an
> earlier mail. Simply removing the DECLARE_UNO3_AGG_DEFAULTS(...) removes
> forwarding of acquire() and release() calls to the baseclass, I don't
> think that's correct. See comphelper/inc/comphelper/uno3.hxx
Is it wise to fix the code for gcc 4.6 *now*? Every fix might introduce
new bugs and gcc 4.6 is not the "standard" compiler nowadays (AFAIK).
Regards,
Mathias