da-viper wrote:

> This change looks fine but I wonder whether we should discard the entry when 
> the value is null rather than setting an empty string.

Updated. It seems this is a reoccurrence in the API that `const char *` gets 
converted to `std::string` without a check or stringRef conversion. 

https://github.com/llvm/llvm-project/pull/218906
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to