<table><tr><td style="">davidhurka 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/D20868">View Revision</a></tr></table><br /><div><div><p>How about the following approach (not tested)? This way, other parameters (like <tt style="background: #ebebeb; font-size: 13px;">nameddest</tt>) wouldn’t break it and can be added later.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="cpp" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">
<span style="color: #aa4000">const</span> <span class="n">QString</span> <span class="n">fragment</span> <span style="color: #aa2211">=</span> <span class="n">url</span><span class="p">.</span><span class="n">fragment</span><span class="p">(</span><span class="n">QUrl</span><span style="color: #aa2211">::</span><span class="n">EncodeDelimiters</span><span class="p">);</span>

<span style="color: #aa4000">bool</span> <span class="n">gotOpenPageNumber</span> <span style="color: #aa2211">=</span> <span style="color: #304a96">false</span><span class="p">;</span>
<span style="color: #aa4000">int</span> <span class="n">openPageNumber</span> <span style="color: #aa2211">=</span> <span class="n">fragment</span><span class="p">.</span><span class="n">toInt</span><span class="p">(</span><span style="color: #aa2211">&</span><span class="n">gotOpenPageNumber</span><span class="p">);</span>

<span style="color: #aa4000">for</span> <span class="p">(</span> <span class="n">QString</span> <span style="color: #a0a000">parameter</span> <span class="p">:</span> <span class="n">fragment</span><span class="p">.</span><span class="n">split</span><span class="p">(</span><span class="n">QChar</span><span class="p">(</span><span style="color: #766510">'&'</span><span class="p">)</span> <span class="p">)</span>
<span class="p">{</span>
    <span style="color: #aa4000">if</span> <span class="p">(</span> <span class="n">parameter</span><span class="p">.</span><span class="n">startsWith</span><span class="p">(</span><span style="color: #766510">"page="</span><span class="p">,</span> <span class="n">Qt</span><span style="color: #aa2211">::</span><span class="n">CaseInsensitive</span><span class="p">)</span> <span class="p">)</span>
    <span class="p">{</span>
        <span style="color: #aa4000">bool</span> <span class="n">ok</span> <span style="color: #aa2211">=</span> <span style="color: #304a96">false</span><span class="p">;</span>
        <span style="color: #aa4000">int</span> <span class="n">page</span> <span style="color: #aa2211">=</span> <span class="n">parameter</span><span class="p">.</span><span class="n">mid</span><span class="p">(</span><span style="color: #601200">5</span><span class="p">).</span><span class="n">toInt</span><span class="p">(</span><span style="color: #aa2211">&</span><span class="n">ok</span><span class="p">);</span>
        <span style="color: #aa4000">if</span> <span class="p">(</span> <span class="n">ok</span> <span class="p">)</span>
        <span class="p">{</span>
            <span class="n">openPageNumber</span> <span style="color: #aa2211">=</span> <span class="n">page</span><span class="p">;</span>
            <span class="n">gotOpenPageNumber</span> <span style="color: #aa2211">=</span> <span style="color: #304a96">true</span><span class="p">;</span>
        <span class="p">}</span>
    <span class="p">}</span>
<span class="p">}</span></pre></div>

<p>I`m not sure whether QUrl::EncodeDelimiters is optimal.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20868">https://phabricator.kde.org/D20868</a></div></div><br /><div><strong>To: </strong>michaelweghorn, Okular<br /><strong>Cc: </strong>davidhurka, okular-devel, joaonetto, tfella, ngraham, darcyshen, aacid<br /></div>