<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">I am not sure about the logic of the test case, but I do remember that we removed the “expr” alias a few days ago.<div>The rationale was that having both expression and exp made commands like</div><div>(lldb) exp 1 + 2</div><div>ambiguous (is exp == expr or == expression) for no good actual reason - and we avoided tackling the general issue of checking for “alias equality” as a potential ambiguity solver.</div><div>Just a data point - not sure how helpful.<br><div><br><div>
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div style="font-size: medium; orphans: 2; widows: 2; border-collapse: separate; border-spacing: 0px;"><span style="font-size: 12px; orphans: auto; widows: auto;">Enrico Granata</span><br style="font-size: 12px; orphans: auto; widows: auto;"><span style="font-size: 12px; orphans: auto; widows: auto;">✉ egranata@</span><font color="#ff2600" style="font-size: 12px; orphans: auto; widows: auto;"></font><span style="font-size: 12px; orphans: auto; widows: auto;">.com</span><br style="font-size: 12px; orphans: auto; widows: auto;"><span style="font-size: 12px; orphans: auto; widows: auto;">✆ 27683</span></div></div>
</div>
<br><div><div>On Apr 23, 2013, at 12:05 PM, "Kopec, Matt" <<a href="mailto:matt.kopec@intel.com">matt.kopec@intel.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Sorry, I should have mentioned it in the commit message.<br><br>There was a failure in TestAliases.py where if you alias the command 'expr' twice, the second alias fails because the command interpreter failed to return the CommandSP for 'expr'. What actually happened is GetCommandSP returns a list of matches (2 in this case) if there is more than one match. So it would find 'expr' and the first alias for expr. I added a call to only return CommandSP for command (if one is found, since that is the one I want) and then search aliases after if nothing could be found.<br><br>Does that seem reasonable?<br><br>Thanks,<br>Matt<br><br>On 2013-04-23, at 2:53 PM, <<a href="mailto:jingham@apple.com">jingham@apple.com</a>><br>wrote:<br><br><blockquote type="cite">Matt,<br><br>What was this fixing?<br><br>Jim<br><br>On Apr 23, 2013, at 9:17 AM, Matt Kopec <<a href="mailto:Matt.Kopec@intel.com">Matt.Kopec@intel.com</a>> wrote:<br><br><blockquote type="cite">Author: mkopec<br>Date: Tue Apr 23 11:17:32 2013<br>New Revision: 180117<br><br>URL: <a href="http://llvm.org/viewvc/llvm-project?rev=180117&view=rev">http://llvm.org/viewvc/llvm-project?rev=180117&view=rev</a><br>Log:<br>Look for an inexact match in just the commands before searching the alias commands as well.<br><br><br>Modified:<br> lldb/trunk/source/Interpreter/CommandInterpreter.cpp<br><br>Modified: lldb/trunk/source/Interpreter/CommandInterpreter.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/CommandInterpreter.cpp?rev=180117&r1=180116&r2=180117&view=diff">http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/CommandInterpreter.cpp?rev=180117&r1=180116&r2=180117&view=diff</a><br>==============================================================================<br>--- lldb/trunk/source/Interpreter/CommandInterpreter.cpp (original)<br>+++ lldb/trunk/source/Interpreter/CommandInterpreter.cpp Tue Apr 23 11:17:32 2013<br>@@ -848,10 +848,15 @@ CommandInterpreter::GetCommandObject (co<br> command_obj = GetCommandSP (cmd_cstr, true, true, matches).get();<br> }<br><br>- // Finally, if there wasn't an exact match among the aliases, look for an inexact match<br>- // in both the commands and the aliases.<br>+ // If there wasn't an exact match among the aliases, look for an inexact match<br>+ // in just the commands.<br><br> if (command_obj == NULL)<br>+ command_obj = GetCommandSP(cmd_cstr, false, false, matches).get();<br>+<br>+ // Finally, if there wasn't an inexact match among the commands, look for an inexact<br>+ // match in both the commands and aliases.<br>+ if (command_obj == NULL)<br> command_obj = GetCommandSP(cmd_cstr, true, false, matches).get();<br><br> return command_obj;<br><br><br>_______________________________________________<br>lldb-commits mailing list<br><a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits<br></blockquote><br></blockquote><br><br>_______________________________________________<br>lldb-commits mailing list<br><a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</div></blockquote></div><br></div></div></body></html>