Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | OpenOffice++ (extensions, svtools) | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | utilities | Reporter: | Daniel Darabos <darabos.daniel> | ||||||
Component: | code | Assignee: | carsten.driesner | ||||||
Status: | CLOSED FIXED | QA Contact: | Unknown <non-migrated> | ||||||
Severity: | Trivial | ||||||||
Priority: | P3 | CC: | clippka, frank.schoenheit, issues | ||||||
Version: | 680 | ||||||||
Target Milestone: | OOo 2.3 | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Issue Type: | PATCH | Latest Confirmation in: | --- | ||||||
Developer Difficulty: | --- | ||||||||
Issue Depends on: | |||||||||
Issue Blocks: | 73468 | ||||||||
Attachments: |
|
Description
Daniel Darabos
2007-05-17 00:35:33 UTC
Created attachment 45163 [details]
Patches for extensions
Created attachment 45164 [details]
Patches for svtools
cd->cyhawk: Thanks for your patches. Please give me some time to check them. I have no objections on the patch for the unoimap.cxx file. CL->cyhawk: Can you explain why you changed the default to use IMAP_OBJ_POLYGON instead of IMAP_OBJ_RECTANGLE for unknown imap object types? cd->fs: Could you please check the changes within extensions. You are the main developer of this code. Thanks. fs->cyhawk: I'm not sure about the motivation of the patch in extensions/source/propctrlr/propertyhandler.hxx. I guess it's about m_aMutex (not) being the very first member, which might be desirable in case it's used as reference to the ctor of other members. Is this true? If so, I'd prefer deriving PropertyHandler from ::cppu::BaseMutex, which would reach the same goal in a more robust way. cd->cyhawk: We have some comments about your changes. Please try to answer them. I have accepted all changes that are not controversial. I am going to use this issue to commit them. Without a comment from your side all other changes won't be committed. cd: No answer from patch submitter. Therefore controversial parts of the patch omitted. cd: Verified. cd: Verified on master. Sorry, I don't know why I didn't notice your questions regarding this patch. I have now checked propertyhandler.cxx and indeed m_aPropertyListeners (declared after m_aMutex in propertyhandler.hxx) gets m_aMutex as a parameter to its constructor during initialization. Would you like me to submit a new patch deriving PropertyHandler from BaseMutex? would be great, thanks. |