Remove use of compiler extension in LC table file definitions

[jphickey]

Checklist (Please check before submitting)

• [x] I reviewed the Contributing Guide.
• [x] I reviewed the README file to see if the feature is in the major future work.
• [x] I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe. The __attribute__((__used__)) is a GNU compiler extension flag and such constructs should be avoided in portable code.

Describe the solution you'd like Remove

Additional context This flag is only in there to avoid a warning due to the use of static - but this object should not be static to begin with, then everything works without the use of special flags.

Requester Info Joseph Hickey, Vantage Systems, Inc.

[jphickey]

There is a additional inconsistency in the table defs in that the tables have the word "Default" baked into their name -- LC_DefaultADT and LC_DefaultWDT. In reality the tables are called LC_ADT and LC_WDT, respectively. These just happen to be the default values for those tables (that's OK), but the runtime name of the table should not change based on whether it is the default value or not.