[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-edu-devel
Subject:    D17201: Fix for memory leak in SessionManagerFixed
From:       Hartmut Riesenbeck <noreply () phabricator ! kde ! org>
Date:       2018-11-27 20:20:58
Message-ID: 754f9badf639ac675b778aa48cf5e1cf () localhost ! localdomain
[Download RAW message or body]

[Attachment #2 (text/plain)]

hriesenbeck created this revision.
Herald added a project: KDE Edu.
Herald added a subscriber: kde-edu.
hriesenbeck requested review of this revision.

REVISION SUMMARY
  An already by Inge Wallin marked memory leak is fixed by removing the
  choosen test entries from the source list after copying them into the
  destination list. So at the end of the choosing process only unused
  entries remaining in the source list which are deleted before reusing
  the list.
  
  Depends on D17180 <https://phabricator.kde.org/D17180>

REPOSITORY
  R340 Parley

BRANCH
  fix_memory_leak_session_manager_for_review (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D17201

AFFECTED FILES
  src/practice/sessionmanagerfixed.cpp

To: hriesenbeck
Cc: kde-edu, narvaez, apol

[Attachment #3 (text/html)]

<table><tr><td style="">hriesenbeck created this revision.<br />Herald added a \
project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />hriesenbeck requested \
review of this revision. </td><a style="text-decoration: none; padding: 4px 8px; \
margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: \
3px; background-color: #F7F7F9; background-image: linear-gradient(to \
bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" \
href="https://phabricator.kde.org/D17201">View Revision</a></tr></table><br \
/><div><strong>REVISION SUMMARY</strong><div><p>An already by Inge Wallin marked \
memory leak is fixed by removing the<br /> choosen test entries from the source list \
after copying them into the<br /> destination list. So at the end of the choosing \
process only unused<br /> entries remaining in the source list which are deleted \
before reusing<br /> the list.</p>

<p>Depends on <a href="https://phabricator.kde.org/D17180" style="background-color: \
#e7e7e7;  border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17180</a></p></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R340 Parley</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>fix_memory_leak_session_manager_for_review \
(branched from master)</div></div></div><br /><div><strong>REVISION \
DETAIL</strong><div><a \
href="https://phabricator.kde.org/D17201">https://phabricator.kde.org/D17201</a></div></div><br \
/><div><strong>AFFECTED \
FILES</strong><div><div>src/practice/sessionmanagerfixed.cpp</div></div></div><br \
/><div><strong>To: </strong>hriesenbeck<br /><strong>Cc: </strong>kde-edu, narvaez, \
apol<br /></div>



[prev in list] [next in list] [prev in thread] [next in thread]