| Message ID | 20240222185907.4078238-1-dm.chestnykh@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [uclibc-ng-devel] ldso: Remove unneeded semicolons. | expand |
Hi Dmitry, Dmitry Chestnykh wrote, > We use semicolons in the place of > `DL_RELOCATE_RELR()` and `DL_DO_RELOCATE_RELR()` 'calling' > so the semicolon in the macro definition leads to > semicolon duplication after preprocessing. > > Signed-off-by: Dmitry Chestnykh <dm.chestnykh@gmail.com> Applied and pushed, thx, Waldemar
diff --git a/ldso/include/dl-elf.h b/ldso/include/dl-elf.h index 7143b572c..832f6b1dd 100644 --- a/ldso/include/dl-elf.h +++ b/ldso/include/dl-elf.h @@ -274,7 +274,7 @@ unsigned int __dl_parse_dynamic_info(ElfW(Dyn) *dpnt, unsigned long dynamic_info relr_reloc_addr += CHAR_BIT * sizeof(ElfW(Relr)) - 1; \ } \ } \ - } while (0); + } while (0) /* The macro to prepare data for the above DL_DO_RELOCATE_RELR */ #define DL_RELOCATE_RELR(dyn) \ @@ -289,7 +289,7 @@ unsigned int __dl_parse_dynamic_info(ElfW(Dyn) *dpnt, unsigned long dynamic_info dyn->libname, (void *)relr_start, (void *)relr_end); \ DL_DO_RELOCATE_RELR(dyn->loadaddr, relr_start, relr_end); \ } \ - } while (0); + } while (0) #endif /* __FDPIC__ */ #endif /* _DL_ELF_H */
We use semicolons in the place of `DL_RELOCATE_RELR()` and `DL_DO_RELOCATE_RELR()` 'calling' so the semicolon in the macro definition leads to semicolon duplication after preprocessing. Signed-off-by: Dmitry Chestnykh <dm.chestnykh@gmail.com> --- ldso/include/dl-elf.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)