Re: [Lldb-commits] [PATCH] D56458: Fix unused private field warning.

2019-01-13 Thread David Blaikie via lldb-commits
Might be better to only define the variable if HAVE_LIBCOMPRESSION is defined? On Wed, Jan 9, 2019 at 8:58 AM Raphael Isemann via Phabricator via llvm-commits wrote: > This revision was not accepted when it landed; it landed in state "Needs > Review". > This revision was automatically updated

[Lldb-commits] [PATCH] D56458: Fix unused private field warning.

2019-01-08 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL350675: Fix unused private field warning. (authored by teemperor, committed by ). Herald added a subscriber:

[Lldb-commits] [PATCH] D56458: Fix unused private field warning.

2019-01-08 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor created this revision. Herald added a subscriber: lldb-commits. The member is private and unused if HAVE_LIBCOMPRESSION is undefined, which triggers Clang's -Wunused-private-field warning. Repository: rLLDB LLDB https://reviews.llvm.org/D56458 Files: