Skip to content

Commit b399885

Browse files
committed
clang-tidy: use auto
Found with modernize-use-auto Signed-off-by: Rosen Penev <[email protected]>
1 parent 3f91ab2 commit b399885

File tree

1 file changed

+15
-15
lines changed

1 file changed

+15
-15
lines changed

src/patchelf.cc

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,7 @@ ElfType getElfType(const FileContents & fileContents)
373373

374374
static void checkPointer(const FileContents & contents, void * p, unsigned int size)
375375
{
376-
unsigned char * q = static_cast<unsigned char *>(p);
376+
auto q = static_cast<unsigned char *>(p);
377377
if (!(q >= contents->data() && q + size <= contents->data() + contents->size()))
378378
error("data region extends past file end");
379379
}
@@ -648,7 +648,7 @@ template<ElfFileParams>
648648
std::string & ElfFile<ElfFileParamNames>::replaceSection(const SectionName & sectionName,
649649
unsigned int size)
650650
{
651-
ReplacedSections::iterator i = replacedSections.find(sectionName);
651+
auto i = replacedSections.find(sectionName);
652652
std::string s;
653653

654654
if (i != replacedSections.end()) {
@@ -1084,7 +1084,7 @@ void ElfFile<ElfFileParamNames>::rewriteHeaders(Elf_Addr phdrAddress)
10841084
(e.g., those produced by klibc's klcc). */
10851085
auto shdrDynamic = findSection2(".dynamic");
10861086
if (shdrDynamic) {
1087-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic->sh_offset));
1087+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic->sh_offset));
10881088
unsigned int d_tag;
10891089
for ( ; (d_tag = rdi(dyn->d_tag)) != DT_NULL; dyn++)
10901090
if (d_tag == DT_STRTAB)
@@ -1135,7 +1135,7 @@ void ElfFile<ElfFileParamNames>::rewriteHeaders(Elf_Addr phdrAddress)
11351135
if (rdi(shdrs[i].sh_type) != SHT_SYMTAB && rdi(shdrs[i].sh_type) != SHT_DYNSYM) continue;
11361136
debug("rewriting symbol table section %d\n", i);
11371137
for (size_t entry = 0; (entry + 1) * sizeof(Elf_Sym) <= rdi(shdrs[i].sh_size); entry++) {
1138-
Elf_Sym * sym = (Elf_Sym *) (contents + rdi(shdrs[i].sh_offset) + entry * sizeof(Elf_Sym));
1138+
auto sym = (Elf_Sym *)(contents + rdi(shdrs[i].sh_offset) + entry * sizeof(Elf_Sym));
11391139
unsigned int shndx = rdi(sym->st_shndx);
11401140
if (shndx != SHN_UNDEF && shndx < SHN_LORESERVE) {
11411141
if (shndx >= sectionsByOldIndex.size()) {
@@ -1185,7 +1185,7 @@ void ElfFile<ElfFileParamNames>::modifySoname(sonameMode op, const std::string &
11851185
char * strTab = (char *) contents + rdi(shdrDynStr.sh_offset);
11861186

11871187
/* Walk through the dynamic section, look for the DT_SONAME entry. */
1188-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic.sh_offset));
1188+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic.sh_offset));
11891189
Elf_Dyn * dynSoname = nullptr;
11901190
char * soname = nullptr;
11911191
for ( ; rdi(dyn->d_tag) != DT_NULL; dyn++) {
@@ -1286,7 +1286,7 @@ void ElfFile<ElfFileParamNames>::modifyRPath(RPathOp op,
12861286
generates a DT_RPATH and DT_RUNPATH pointing at the same
12871287
string. */
12881288
std::vector<std::string> neededLibs;
1289-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic.sh_offset));
1289+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic.sh_offset));
12901290
Elf_Dyn *dynRPath = nullptr, *dynRunPath = nullptr;
12911291
char * rpath = nullptr;
12921292
for ( ; rdi(dyn->d_tag) != DT_NULL; dyn++) {
@@ -1368,7 +1368,7 @@ void ElfFile<ElfFileParamNames>::modifyRPath(RPathOp op,
13681368
return;
13691369
}
13701370

1371-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic.sh_offset));
1371+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic.sh_offset));
13721372
Elf_Dyn * last = dyn;
13731373
for ( ; rdi(dyn->d_tag) != DT_NULL; dyn++) {
13741374
if (rdi(dyn->d_tag) == DT_RPATH) {
@@ -1465,7 +1465,7 @@ void ElfFile<ElfFileParamNames>::removeNeeded(const std::set<std::string> & libs
14651465
auto shdrDynStr = findSection(".dynstr");
14661466
char * strTab = (char *) contents + rdi(shdrDynStr.sh_offset);
14671467

1468-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic.sh_offset));
1468+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic.sh_offset));
14691469
Elf_Dyn * last = dyn;
14701470
for ( ; rdi(dyn->d_tag) != DT_NULL; dyn++) {
14711471
if (rdi(dyn->d_tag) == DT_NEEDED) {
@@ -1493,7 +1493,7 @@ void ElfFile<ElfFileParamNames>::replaceNeeded(const std::map<std::string, std::
14931493
auto shdrDynStr = findSection(".dynstr");
14941494
char * strTab = (char *) contents + rdi(shdrDynStr.sh_offset);
14951495

1496-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic.sh_offset));
1496+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic.sh_offset));
14971497

14981498
unsigned int verNeedNum = 0;
14991499

@@ -1551,7 +1551,7 @@ void ElfFile<ElfFileParamNames>::replaceNeeded(const std::map<std::string, std::
15511551

15521552
unsigned int verStrAddedBytes = 0;
15531553

1554-
Elf_Verneed * need = (Elf_Verneed *) (contents + rdi(shdrVersionR.sh_offset));
1554+
auto need = (Elf_Verneed *)(contents + rdi(shdrVersionR.sh_offset));
15551555
while (verNeedNum > 0) {
15561556
char * file = verStrTab + rdi(need->vn_file);
15571557
auto i = libs.find(file);
@@ -1649,8 +1649,8 @@ void ElfFile<ElfFileParamNames>::noDefaultLib()
16491649
{
16501650
auto shdrDynamic = findSection(".dynamic");
16511651

1652-
Elf_Dyn * dyn = (Elf_Dyn *) (contents + rdi(shdrDynamic.sh_offset));
1653-
Elf_Dyn * dynFlags1 = nullptr;
1652+
auto dyn = (Elf_Dyn *)(contents + rdi(shdrDynamic.sh_offset));
1653+
auto dynFlags1 = (Elf_Dyn *)nullptr;
16541654
for ( ; rdi(dyn->d_tag) != DT_NULL; dyn++) {
16551655
if (rdi(dyn->d_tag) == DT_FLAGS_1) {
16561656
dynFlags1 = dyn;
@@ -1692,9 +1692,9 @@ void ElfFile<ElfFileParamNames>::clearSymbolVersions(const std::set<std::string>
16921692
auto shdrDynsym = findSection(".dynsym");
16931693
auto shdrVersym = findSection(".gnu.version");
16941694

1695-
char * strTab = (char *) contents + rdi(shdrDynStr.sh_offset);
1696-
Elf_Sym * dynsyms = (Elf_Sym *) (contents + rdi(shdrDynsym.sh_offset));
1697-
Elf_Versym * versyms = (Elf_Versym *) (contents + rdi(shdrVersym.sh_offset));
1695+
auto strTab = (char *)contents + rdi(shdrDynStr.sh_offset);
1696+
auto dynsyms = (Elf_Sym *)(contents + rdi(shdrDynsym.sh_offset));
1697+
auto versyms = (Elf_Versym *)(contents + rdi(shdrVersym.sh_offset));
16981698
size_t count = rdi(shdrDynsym.sh_size) / sizeof(Elf_Sym);
16991699

17001700
if (count != rdi(shdrVersym.sh_size) / sizeof(Elf_Versym))

0 commit comments

Comments
 (0)