<table><tr><td style="">brauch edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-3pt3bouuqdfopzc/">(Show Details)</a>
</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/D14826">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This is a slightly modified version of the InlineNote interface written by Michal in D12662.<br />
<br />
Changes:<br />
 - move the interface to the view instead of the document (rather trivial)<br />
 - rework how notes are queried (see below)<br />
<br />
I'm not sure about which API is better, the old suggestion or this one. The problem I see with the old one is that you need to return pointers to instances of the notes, which you need to memory-manage yourself. This means you either allocate all of them immediately, or you need some rather complex tracking of what instances you created and when to free them again.<br />
<br />
Effectively, I tried to implement inline error display (see attachments below) for KDevelop, once with the API from the other patch and once with this one. The amount of lines is approximately the same, but with this API, you can implement it only as a view on top of what's already there, i.e. you only add three member functions which return things based on the existing data. With the other API, you need a vector as member which contains all the notes instances and you need to manage that.<br />
<br />
What do you think? I'd especially be interested in Michal's thoughts :)</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14826">https://phabricator.kde.org/D14826</a></div></div><br /><div><strong>To: </strong>brauch, michalsrb, dhaumann, cullmann<br /><strong>Cc: </strong>anthonyfieroni, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>