hapeeeeee wrote:

> It seems wrong to have the source manager pretend there are more lines in the 
> file with the content "there are no more lines in this file". At some point, 
> someone is going to want to know that there are no more lines and this 
> artificial content will be confusing.
> 
> It would be better to have the source manager return some kind of error and 
> have the printer cons up the user-facing message.

I believe you are correct. I have updated the code so that the prompt output is 
now returned by `SourceManager`, and I have differentiated the causes. Could 
you please review my commit again? @jimingham 



https://github.com/llvm/llvm-project/pull/137515
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to