Mailing-List: contact ooo-issues-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: ooo-issues@incubator.apache.org
From: bugzilla@apache.org
To: ooo-issues@incubator.apache.org
Subject: [Bug 113608] animations:  All animation nodes are leaked
Date: Fri, 15 Jun 2012 09:39:52 +0000
Message-ID: <bug-113608-248469-RBZ5D6whLp@https.issues.apache.org/ooo/>
In-Reply-To: <bug-113608-248469@https.issues.apache.org/ooo/>
References: <bug-113608-248469@https.issues.apache.org/ooo/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Auto-Submitted: auto-generated
MIME-Version: 1.0

https://issues.apache.org/ooo/show_bug.cgi?id=113608

--- Comment #11 from Andre <af@a-w-f.de> ---
Well, as long as SdrObject and SvxShape are two separate classes, everything
remains a workaround.  And as long an SdrObject holds its SvxShape as weak
pointer, every access to that weak pointer has to be guarded.  The only strange
thing is that it appears to be (but I did not check that) that the reference
count of the weak pointer goes to zero without the SvxShape being destroyed.

It might be a good idea to remove the mpSvxShape member altogether.

Regarding you proposed change:

 if( !HasSdrObjectOwnership() ){
    mpObj->setUnoShape( NULL, SdrObject::GrantXShapeAccess()
    ...

I am not sure if it is valid to modify mpObj when it is not owned by the
SvxShape ie HasSdrObjectOwnership() is false.  It might be OK to make the
setUnoShape(NULL...) call when mpObj points back to "this".

-- 
You are receiving this mail because:
You are on the CC list for the bug.