On Mon, Jul 3, 2023, at 10:57 AM, Ashutosh Bapat wrote: > On Mon, Jul 3, 2023 at 6:52 PM Ashutosh Bapat > <ashutosh.bapat....@gmail.com> wrote: > > > > The switch is on action which is an enum. So without default it will > > provide a compilation warning for missing enums. Adding "default" case > > defeats that purpose. I think we should just return "???" from outside > > switch block. > >
Yeah. I don't think any gcc -Wswitch options have effect if a default is used so your suggestion is good for wrong/missing message types in the future. > PFA patch. WFM. -- Euler Taveira EDB https://www.enterprisedb.com/