[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-11-01 Thread jsk
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457


User jsk changed the following:

What|Old value |New value

  Status|VERIFIED  |CLOSED





--- Additional comments from [EMAIL PROTECTED] Thu Nov  1 08:57:03 + 
2007 ---
Rechecked in OOG680m7, closing

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-08-02 Thread jsk
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457


User jsk changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from [EMAIL PROTECTED] Thu Aug  2 13:29:34 + 
2007 ---
The office (soffice.bin) stays at 15656KB after several runs of the testcode.
Veriffied.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-07-26 Thread ab
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457


User ab changed the following:

What|Old value |New value

 Assigned to|ab|jsk





--- Additional comments from [EMAIL PROTECTED] Thu Jul 26 08:50:27 + 
2007 ---
ab-jsk: To verify just run the test code some times. With the ab34 Office
memorey usage before and after the run should be nearly the same,
whereas in the m222 master the memery use increases significantly
with every run.


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-05-06 Thread kpalagin
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457





--- Additional comments from [EMAIL PROTECTED] Sun May  6 10:09:46 + 
2007 ---
terrye,
I would very much like to see OpenOffice to have better performance.
If you see potential for improvement please file new issue - I am sure 
community will support new RFE.
Thank you very much!
WBR
Kirill Palagin.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-05-05 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457





--- Additional comments from [EMAIL PROTECTED] Sun May  6 00:48:57 + 
2007 ---
I've done quite a lot of work on the analysis here, but it's the old problem --
finding the hours in the day to move this one forward.   If you feel that we
should make this publicly available then we should open another issue, but this
doesn't relate to 73457 -- which was a specific leak and bug fix.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-04-29 Thread kpalagin
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457





--- Additional comments from [EMAIL PROTECTED] Sun Apr 29 06:14:01 + 
2007 ---
I remember seeing the idea of code refactoring to improve performance, but 
can't find anymore in this discussion.
Do we have issue for that?

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-03-12 Thread ab
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457


User ab changed the following:

What|Old value |New value

  Status|STARTED   |RESOLVED

  Resolution|  |FIXED





--- Additional comments from [EMAIL PROTECTED] Mon Mar 12 12:19:16 + 
2007 ---
ab-Terry: Thanks for your great analysis. I've checked in your
minimum impact patch now.

- FIXED


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-02-21 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457


User terrye changed the following:

What|Old value |New value

 Attachment data|  |Created an attachment (id=
|  |43254)
Bug Analysis Workin
|  |g Paper






--- Additional comments from [EMAIL PROTECTED] Thu Feb 22 00:16:26 + 
2007 ---
Created an attachment (id=43254)
Bug Analysis Working Paper


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-02-10 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457


User terrye changed the following:

What|Old value |New value

 Summary|Memory Leak in Basic Argum|Memory Leak in all Basic t
|ent Handling  |ype void Method calls





--- Additional comments from [EMAIL PROTECTED] Sat Feb 10 12:44:14 + 
2007 ---
*** FIX ***

The minimum impact change which removes this leak is a one line change to
sbxmod.cxx:
@@ -2057,7 +2057,8 @@
if( mpPar.Is() )
 {
// this, als Element 0 eintragen, aber den Parent nicht
umsetzen!
-   mphoPar-PutDirect( pThisCopy, 0 );
+   if (GetType() != SbxVOID) mpPar-PutDirect( pThisCopy, 0 );
SetParameters( NULL );
 }

This change should be put in ASAP in 2.3 or 2.4, as it will close a major memory
leak in all applications which make heavy use of Basic.

Note that this whole code area has a number of implementation weaknesses that
impact heavily on runtime performance.  I've drafted a paper on this and sent to
AB for comment. I will attach it as background once AB has had a chance to 
comment.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[api-issues] [Issue 73457] Memory Leak in all Basic t ype void Method calls

2007-02-10 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=73457





--- Additional comments from [EMAIL PROTECTED] Sat Feb 10 12:54:32 + 
2007 ---
By the way, the diff header line should read @@ -2057,7 +2057,7 @@

I removed a copy of my audit comment and forget to change the line count.  Also
the line number 2057 refers to the ooo-build 2.0.4 variant.  Sorry.  //Terry  

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]