<table><tr><td style="">asemke added a comment.
</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/D15151">View Revision</a></tr></table><br /><div><div><p>This makes sense. But let's unify and simplify the parameter naming a bit and let's call the additional new parameter in the constructor simply "internal". The parameter is called "internal" and the functions to get and to set it are called isInternal() and setInternal(), respectively. This would also be similar to the current naming in Session::setInternal().</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R55 Cantor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15151">https://phabricator.kde.org/D15151</a></div></div><br /><div><strong>To: </strong>sirgienko, asemke, filipesaraiva<br /><strong>Cc: </strong>filipesaraiva, kde-edu, asemke, narvaez, apol<br /></div>