[Yade-dev] [Bug 759481] Re: Possible "wrong memory access" in many places

2011-10-12 Thread Anton Gladky
** Changed in: yade Status: New => Invalid -- You received this bug notification because you are a member of Yade developers, which is the registrant for Yade. https://bugs.launchpad.net/bugs/759481 Title: Possible "wrong memory access" in many places Status in Yet Another Dynamic Engi

[Yade-dev] [Bug 759481] Re: Possible "wrong memory access" in many places

2011-04-13 Thread Anton Gladky
It seems, if (!B) continue; is already added everywhere, where it is necessary. Jan, according to Sergei's message [1], could you, please, delete "if(!I) continue;" from your code and check, whether the bug appears. If not, it is better to delete this "if" from the code. Anton [1] http://w

Re: [Yade-dev] [Bug 759481] Re: Possible "wrong memory access" in many places

2011-04-13 Thread Anton Gladky
So, you mean, that we need to add only: if (!B) continue; right? And there is no sense for interactions. Anton On Wed, Apr 13, 2011 at 9:16 AM, Sergei D. wrote: > 13.04.2011 10:36, Anton Gladky: >> >> Is it ok for everybody to fix the code by this way? >> There is almost everywhere "if(!I->is

Re: [Yade-dev] [Bug 759481] Re: Possible "wrong memory access" in many places

2011-04-13 Thread Sergei D.
13.04.2011 10:36, Anton Gladky: Is it ok for everybody to fix the code by this way? There is almost everywhere "if(!I->isReal()) continue;", but it insufficient, probably... This is not a case. If you look at BodyContainer::erase and InteractionContainer::erase you will see differencies. Fo

[Yade-dev] [Bug 759481] Re: Possible "wrong memory access" in many places

2011-04-12 Thread Anton Gladky
** Attachment added: "Output of "grep -nHIir -- 'FOREACH(const shared_ptrgrep_I.output" command" https://bugs.launchpad.net/bugs/759481/+attachment/2028449/+files/grep_I.output -- You received this bug notification because you are a member of Yade developers, which is the registrant for Yade.

[Yade-dev] [Bug 759481] Re: Possible "wrong memory access" in many places

2011-04-12 Thread Anton Gladky
Is it ok for everybody to fix the code by this way? There is almost everywhere "if(!I->isReal()) continue;", but it insufficient, probably... -- You received this bug notification because you are a member of Yade developers, which is the registrant for Yade. https://bugs.launchpad.net/bugs/75948