Add include guard to hash_table.h

beehive-lab / mambo

A low-overhead dynamic binary instrumentation and modification tool for ARM (both AArch32 and AArch64 support) and RISC-V (RV64GC).

Apache License 2.0

318 stars 69 forks source link

Add include guard to hash_table.h #117

Closed AlphaDaze closed 5 months ago

AlphaDaze commented 6 months ago

Add guards to the mambo's hash_table header to avoid double inclusion.

Required for plugins that utilise it over multiple files.

IgWod commented 6 months ago

Hi,

Thank you for your contribution. It looks good to me. Since we don't have a CI, I want to build it first on RISC-V before merging just in case - we had some issues in the past when we tested only one architecture, so better to be safe! But, anyway, nothing for you to worry about. It should be merged in the next few days.

IgWod commented 5 months ago

I tested it on RISC-V and ARM64 and it seems it caused no compilation issues, as expected. So, I'm merging it now.