<table><tr><td style="">qi437103 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/D2199" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>IMHO, common is just a library that can be used by either GDB or LLDB, and with default settings working out-of-box for GDB. If GDB or LLDB plugin wants different behavior, it could subclass and change it. And it's the GDB or LLDB plugin's responsibility to test whether common works as expected.</p>
<p>Also, technically, you can't just simply test common without pulling in GDB or LLDB specific code, as common itself isn't a complete plugin and isn't associated with any debugger. While it's possible to test it by adding extra mock objects and use either/both GDB and LLDB as backend, I feel it's simpler to just test it in either GDB or LLDB code.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKDEVELOP KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2199" rel="noreferrer">https://phabricator.kde.org/D2199</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>qi437103, KDevelop, apol<br /><strong>Cc: </strong>kdevelop-devel<br /></div>