<div dir="ltr">This seems right:<div><br></div><div><div>python</div><div>Python 2.7.10 (default, Oct 23 2015, 18:05:06)</div><div>[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-700.0.59.5)] on darwin</div><div>Type "help", "copyright", "credits" or "license" for more information.</div><div>>>> s = 'abc'</div><div>>>> 'bc' in 'abc'</div><div>True</div><div>>>> 'bc' in s</div><div>True</div><div>>>> 'clang' in 'clang'</div><div>True</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 14, 2015 at 6:45 PM, Siva Chandra <span dir="ltr"><<a href="mailto:sivachandra@google.com" target="_blank">sivachandra@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Dec 14, 2015 at 6:42 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
> zturner added inline comments.<br>
><br>
> ================<br>
> Comment at: lldb/trunk/packages/Python/lldbsuite/test/lldbtest.py:622<br>
> @@ -621,3 +621,3 @@<br>
>          return list_or_lambda(value)<br>
> -    elif isinstance(list_or_lambda, list):<br>
> -        return list_or_lambda is None or value is None or value in list_or_lambda<br>
> +    elif isinstance(list_or_lambda, list) or isinstance(list_or_lambda, str):<br>
> +        return value is None or value in list_or_lambda<br>
> ----------------<br>
> I mentioned this in the other thread, but I'm not sure we want the string check.  if `list_or_lambda` is a string, then I think it should just check `list_or_lambda == value`.  Otherwise the test only passes when `value` is a character, which is not now we use any of the decorators.<br>
<br>
</span>AFAIK, there is no character type in Python. The code 'value in<br>
list_or_lambda' returns True if |value| is a substring of<br>
|list_or_lambda|. For example, 'db' in 'lldb' evaluates to True.<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>