xant/libhl

reserved identifier violation

Closed this issue · 8 comments

I would like to point out that identifiers like "__tagged_value" and "_HL_HASHTABLE_H_" do eventually not fit to the expected naming convention of the C language standard.
Would you like to adjust your selection for unique names?

xant commented

will fix by removing the underscores from all headers

@elfring are you actually using this package or just making sure it fits a set of style guidelines?

Some uses of double underscores should also be reconsidered, shouldn't it?

Can C standard-compliance help you to reduce the error probability around specific implementation details?

xant commented

isn't that c++ ?

xant commented

but those are macro arguments ... I think there is no point in changing those since their scope is limited to the actual macro implementation. Is this being reported by some analyzer?

Macro arguments are also identifiers. Would you like to clean-up the remaining source code places which tamper still with the reserved name space?