[Lldb-commits] [PATCH] Make sure we perform full pathname matches when displaying source code.

[jingham at apple.com]

I'm not sure about this fix.  The difference between true & false in the last argument to FileSpec::Equal only matters if one or the other paths is just a filename, but the other has a directory.  That shouldn't help when stepping from source file to source file because we'll only know whether the file is actually different if both have full paths.  So I'd want to know who is doing "is this file the same as that one" but only providing a base-name for one of the files.  That seems like the part that is wrong.

Jim


> On Oct 21, 2014, at 1:03 PM, Stephane Sezer <sas at fb.com> wrote:
> 
> Bump.
> 
> Could someone commit this for me? :)
> 
> -- 
> Stephane Sezer
> 
> On Oct 12, 2014, at 11:07 AM, Todd Fiala <tfiala at google.com> wrote:
> 
>> Hey Greg,
>> 
>> Stephane's patch looks reasonable here.  Can you have a quick look at it?  I can get it checked in if it looks okay to you.
>> 
>> Thanks,
>> Todd
>> 
>> On Wed, Oct 8, 2014 at 3:27 PM, Stephane Sezer <sas at fb.com> wrote:
>> When stepping into a function in a different file but with the same
>> basename, we do not reload the source file and end up displaying the
>> wrong snippet of code. This patch forces filename comparisons to be made
>> with a full path, which fixes the issue.
>> 
>> Ran tests on Mac OS.
>> 
>> 
>> -- 
>> Stephane Sezer
>> 
>> 
>> _______________________________________________
>> lldb-commits mailing list
>> lldb-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>> 
>> 
>> 
>> 
>> -- 
>> Todd Fiala |	 Software Engineer |	 tfiala at google.com	
>> 
> 
> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits