<table><tr><td style="">sirgienko added a subscriber: asemke.<br />sirgienko 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/D14005">View Revision</a></tr></table><br /><div><div><p>Hi, thanks for patch.<br />
I am afraid, that <a href="https://phabricator.kde.org/p/filipesaraiva/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@filipesaraiva</a> don't active in Cantor in this moment, so I recommend change <a href="https://phabricator.kde.org/p/filipesaraiva/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@filipesaraiva</a> to <a href="https://phabricator.kde.org/p/asemke/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@asemke</a>: another active Cantor developer.<br />
What about removing this if condtions at all?</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" 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);">-        if(command.endsWith(QLatin1Char('?')) && !command.endsWith(QLatin1String("??")))
-            command=QLatin1String("help(")+command.left(command.size()-1)+QLatin1Char(')');
-        if(command.startsWith(QLatin1Char('?')))
-            command=QLatin1String("help(")+command.mid(1)+QLatin1Char(')');</pre></div>

<p>As I see, this code don't need, because <tt style="background: #ebebeb; font-size: 13px;">sage</tt> understand <tt style="background: #ebebeb; font-size: 13px;">foo?</tt> and <tt style="background: #ebebeb; font-size: 13px;">?foo</tt> as help request, so we don't need change <tt style="background: #ebebeb; font-size: 13px;">foo?</tt> and <tt style="background: #ebebeb; font-size: 13px;">?foo</tt> to <tt style="background: #ebebeb; font-size: 13px;">help(foo)</tt> command. And this also make the source code command (<tt style="background: #ebebeb; font-size: 13px;">foo??</tt>) working.</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/D14005">https://phabricator.kde.org/D14005</a></div></div><br /><div><strong>To: </strong>arojas, filipesaraiva, sirgienko<br /><strong>Cc: </strong>asemke, kde-edu, narvaez, apol<br /></div>