[Lldb-commits] [lldb] [lldb] Scalar::GetValue() should take a Stream by reference (PR #69231)

2023-10-17 Thread Alex Langford via lldb-commits
https://github.com/bulbazord closed https://github.com/llvm/llvm-project/pull/69231 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Scalar::GetValue() should take a Stream by reference (PR #69231)

2023-10-17 Thread Alex Langford via lldb-commits
bulbazord wrote: > There are probably a lot of functions that could be converted from a "Stream > *" to "Stream &". Yes, when I have a spare 5-10 minutes I pick a function and convert it. I don't want to end up with a huge diff that is difficult to merge downstream so I do it piecemeal.

[Lldb-commits] [lldb] [lldb] Scalar::GetValue() should take a Stream by reference (PR #69231)

2023-10-17 Thread Greg Clayton via lldb-commits
clayborg wrote: There are probably a lot of functions that could be converted from a "Stream *" to "Stream &". https://github.com/llvm/llvm-project/pull/69231 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [lldb] [lldb] Scalar::GetValue() should take a Stream by reference (PR #69231)

2023-10-17 Thread Med Ismail Bennani via lldb-commits
https://github.com/medismailben approved this pull request. LGTM! https://github.com/llvm/llvm-project/pull/69231 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Scalar::GetValue() should take a Stream by reference (PR #69231)

2023-10-16 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Alex Langford (bulbazord) Changes This function always expects the pointer to be valid, a reference seems more appropriate. --- Full diff: https://github.com/llvm/llvm-project/pull/69231.diff 5 Files Affected: - (modified)

[Lldb-commits] [lldb] [lldb] Scalar::GetValue() should take a Stream by reference (PR #69231)

2023-10-16 Thread Alex Langford via lldb-commits
https://github.com/bulbazord created https://github.com/llvm/llvm-project/pull/69231 This function always expects the pointer to be valid, a reference seems more appropriate. >From fd70abe32d822be5a791f8650c86ca61a86fa755 Mon Sep 17 00:00:00 2001 From: Alex Langford Date: Mon, 16 Oct 2023