Don't use "operator" as variable name (IDFGH-8000)

[higaski]

The new MCPWM driver uses the C++ keyword operator as variable name.

esp-idf/components/driver/include/driver/mcpwm_types.h

Line 89 in 30e8f19

typedef bool (*mcpwm_brake_event_cb_t)(mcpwm_oper_handle_t operator, const mcpwm_brake_event_data_t *edata, void *user_ctx);

Is this really necessary?
It's already really painful enough to include esp-idf C code into any kind of C++ file because of all the nested structs with designated initializers which isn't supported by C++ the same way.

[0xjakob]

Hi @higaski, sorry for the inconvenience. Sometimes these things slip through because most developers don't use C++, but only C.

Is this really necessary?

I don't think so. We'll check this and probably change the variable name.