search

EarthScope / evalresp

Evaluates instrument responses in FDSN and derivative formats
GNU Lesser General Public License v3.0
12 stars 2 forks source link

Remove _t (in our structures) #37

Closed andrewcooke-isti closed 7 years ago

andrewcooke-isti commented 7 years ago

Since Posix forbids it (although we're strictly not using Posix anyway).

andrewcooke-isti commented 7 years ago

Looks like we ant _s now. See email thread.

andrewcooke-isti commented 7 years ago

Oh, I had that backwards. _s for structs (afaik you typically don't need to name them at all unless you have circular refs) and nothing for the typedef.

andrewcooke-isti commented 7 years ago

ok, this was difficult because without the _t (even including the _s) we get a conflict: evalresp_log is both a function that does the logging and the typedef for the struct that it takes as a first argument.

the best solution i could come up with was to rename the struct to be evalresp_logger.

andrewcooke-isti commented 7 years ago

commit 5a14ed4