RE: Enhanced error message to include hint messages for redundant options error
| От | houzj.fnst@fujitsu.com |
|---|---|
| Тема | RE: Enhanced error message to include hint messages for redundant options error |
| Дата | |
| Msg-id | OS0PR01MB5716ED6E393DDB10D26EBB6094569@OS0PR01MB5716.jpnprd01.prod.outlook.com обсуждение |
| Ответ на | Re: Enhanced error message to include hint messages for redundant options error (Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>) |
| Ответы |
Re: Enhanced error message to include hint messages for redundant options error
|
| Список | pgsql-hackers |
> > > Thanks! The v5 patch looks good to me. Let's see if all agree on the > > > goto duplicate_error approach which could reduce the LOC by ~80. > > > > I think the "goto duplicate_error" approach looks good, it avoids > > duplicating the same error code multiple times. > > Thanks. I will mark the v5 patch "ready for committer" if no one has comments. Hi, I looked into the patch and noticed a minor thing. + return; /* keep compiler quiet */ } I think we do not need the comment here. The compiler seems not require "return" at the end of function when function's return type is VOID. In addition, it seems better to remove these "return;" like what commit "3974c4" did. Best regards, houzj
В списке pgsql-hackers по дате отправления: