<table><tr><td style="">kossebau 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/D15605">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15605#329363" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15605#329363</a>, <a href="https://phabricator.kde.org/p/rjvbb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rjvbb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Would this be OK as a fix?</p></div>
</blockquote>

<p>I cannot tell, as I still haven't found any API docs for astyle, at least one which has a comment about this new method. So I have no idea what contract one should implement for <tt style="background: #ebebeb; font-size: 13px;">ASSourceIterator::getPeekStart()</tt> and what values are expected when. In the code we have imported here there is only a single usage of that call, which is in that assert.<br />
"If you have inherited this class to access the formatter, you will need to add a method similar to getPeekStart() in the ASStreamIterator class in astyle_main.h." from <a href="http://astyle.sourceforge.net/news.html" class="remarkup-link" target="_blank" rel="noreferrer">http://astyle.sourceforge.net/news.html</a> is the only info I have, which leaves lots of room.<br />
Have you seen any docs for that method?</p>

<p>Based on that a fix could also be to always return <tt style="background: #ebebeb; font-size: 13px;">0</tt> ;)</p>

<p>Your proposed fix might work or not, from what I understood on the quick look at astyle code that assert wants to make sure any other peeking has been done, so peekStart == 0 means kind of the no peeking on-going.<br />
In any case that method implementation needs to have some comment that we added ti without really knowing what it should do, so future code readers do not have to scratch their head what we knew they do not.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15605">https://phabricator.kde.org/D15605</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, kfunk<br /><strong>Cc: </strong>kossebau, kfunk, pino, apol, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>