 |
BorlandTalk.com Borland discussion newsgroups
|
| View previous topic :: View next topic |
| Author |
Message |
Peter Feldbaumer Guest
|
Posted: Tue Apr 17, 2007 5:43 pm Post subject: InvokeRegistry-Bug D2007 |
|
|
Jean-Marie,
good to see that some (real old) bugs in the Delphi soap-units finally get
fixed, however at least one bug in InvokeRegistry.pas was missed for the
D2007-release:
in
procedure TSOAPAttachment.SOAPToObject(...)
it SHOULD read
{ Make sure SOAP Runtime does not delete this object }
| Quote: | AttachFound. <<<< DataContext := Nil;
end; |
however it reads only
DataContext := Nil;
which leads to the soap-attachments not being freed correctly! (they live in
the temp-directory forever...)
If I remember correctly this has already been reported some years ago
(however I could'not find it in google-groups right now) - would be nice to
see that fixed in the next (hopefully upcoming) update...
bestr regards,
Peter
--
Peter Feldbaumer
p dot feldbaumer at utanet dot at |
|
| Back to top |
|
 |
Jean-Marie Babet Guest
|
Posted: Thu Apr 19, 2007 2:25 am Post subject: Re: InvokeRegistry-Bug D2007 |
|
|
Hello Peter,
I believe I just saw email a QC on this issue. I suspect you filed it. Thank
you!
Yes, I have vague memories of a few of us talking about finding all these
temp files created by the SOAPAttachment sitting on our machine. I never
delved in the attachment logic as it was handled by someone else. The person
is no longer with us but for D2007 I had my hands full with schema issues
that I never had the time to look at the attachment area.
I've enabled FastMM on our daily unit tests (as there were several memory
issues, specially with dynamic arrays) but currently there's no attachment
unit test: just the samples that ship with the product. I'll remedy this
issue, pull in the QC and look at this as soon as the pressure lowers a
little [I'm diverted on a different product for probably another week or
so].
Cheers,
Bruneau. |
|
| Back to top |
|
 |
Peter Feldbaumer Guest
|
Posted: Thu Apr 19, 2007 3:06 am Post subject: Re: InvokeRegistry-Bug D2007 |
|
|
Hello Bruneau,
(sorry first for the mix-up with the name...)
"Jean-Marie Babet" <bbabet (AT) borland (DOT) com> wrote in message
news:46268cd6$1 (AT) newsgroups (DOT) borland.com...
| Quote: | I believe I just saw email a QC on this issue. I suspect you filed it.
Thank
you!
|
yes I did, because I just couldn't find any real references to this specific
problem...
| Quote: | Yes, I have vague memories of a few of us talking about finding all these
temp files created by the SOAPAttachment sitting on our machine. I never
delved in the attachment logic as it was handled by someone else. The
person
is no longer with us but for D2007 I had my hands full with schema issues
that I never had the time to look at the attachment area.
|
looking back I think I was trying to understand the attachment logic back in
2003 (after getting leak-reports from memcheck) - I'm just not entirely sure
if I found the solution myself or if there were already some ng/qc-postings
about this problem
| Quote: | I've enabled FastMM on our daily unit tests (as there were several memory
issues, specially with dynamic arrays) but currently there's no attachment
unit test: just the samples that ship with the product. I'll remedy this
issue, pull in the QC and look at this as soon as the pressure lowers a
little [I'm diverted on a different product for probably another week or
so].
|
good to hear that...
Enabling mem-checks is a real good idea - it led me to find several problems
in the soap-units (and for most of them solutions posted in ng's or
qc-reports) - unfortunately they never got fixed (bugs were there from at
least D5 up to D2006) - but many of them are fortunately already fixed in
D2007.
Keep up the good work!
--
Peter Feldbaumer
p dot feldbaumer at utanet dot at |
|
| Back to top |
|
 |
|
|
You cannot post new topics in this forum You cannot reply to topics in this forum You cannot edit your posts in this forum You cannot delete your posts in this forum You cannot vote in polls in this forum
|
|