Inconsistent spelling in symbol definition: interp_add_accumulater

[smitelli]

In hardware/interp.h, the definition of the inline function interp_add_accumulater doesn't match the dictionary spelling of "accumulator" used anywhere else in the project or its documentation.

Since changing it could break existing user code, is this the type of thing where there is precedent to keep the incorrect spelling as an alias?

[peterharperuk]

Usually we add a correct version of the function but keep the old version around to maintain compatibility

[lurch]

Yeah the usual approach is to rename the actual function itself, and then add a #define for the old name, so that existing code keeps working; similar to this .

So in your case you'd probably want to add something like:

#define interp_add_accumulater(interp, lane, val) interp_add_accumulator(interp, lane, val)