Skip to content

Commit

Permalink
bin2llvmir: Remove unused private fields _module and _demangler.
Browse files Browse the repository at this point in the history
Based on Clang 7 warnings:

include/retdec/bin2llvmir/providers/names.h:181:17: warning: private field '_module' is not used [-Wunused-private-field]
include/retdec/bin2llvmir/providers/names.h:185:26: warning: private field '_demangler' is not used [-Wunused-private-field]
  • Loading branch information
s3rvac committed Jan 28, 2019
1 parent edbeb82 commit 4c37f60
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
2 changes: 0 additions & 2 deletions include/retdec/bin2llvmir/providers/names.h
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,9 @@ class NameContainer
using ImportOrdMap = std::map<int, std::string>;

private:
llvm::Module* _module = nullptr;
Config* _config = nullptr;
DebugFormat* _debug = nullptr;
FileImage* _image = nullptr;
demangler::CDemangler* _demangler = nullptr;
Lti* _lti = nullptr;

std::map<retdec::utils::Address, Names> _data;
Expand Down
6 changes: 2 additions & 4 deletions src/bin2llvmir/providers/names.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -262,18 +262,16 @@ bool Names::empty() const
//

NameContainer::NameContainer(
llvm::Module* m,
llvm::Module*,
Config* c,
DebugFormat* d,
FileImage* i,
demangler::CDemangler* dm,
demangler::CDemangler*,
Lti* lti)
:
_module(m),
_config(c),
_debug(d),
_image(i),
_demangler(dm),
_lti(lti)
{
initFromConfig();
Expand Down

0 comments on commit 4c37f60

Please sign in to comment.