Re: transformExpr() refactor
От | Neil Conway |
---|---|
Тема | Re: transformExpr() refactor |
Дата | |
Msg-id | 1099011628.21058.15.camel@localhost.localdomain обсуждение исходный текст |
Ответ на | Re: transformExpr() refactor (Tom Lane <tgl@sss.pgh.pa.us>) |
Ответы |
Re: transformExpr() refactor
|
Список | pgsql-patches |
On Fri, 2004-10-29 at 00:17, Tom Lane wrote: > I don't actually find this to be an improvement. What's the point? > Since all the switch arms are independent, you haven't really done > anything at all to improve the comprehensibility of the code... I think the code is more readable this way. The very fact that the switch branches are completely independent is good enough reason to make them distinct functions, IMHO. Breaking 900 lines of code into smaller chunks of code, each of which does a single conceptual task, just makes the whole enterprise easier to understand. As for sharing code between the functions, I agree that isn't done today -- but it will be easier to do in the future with this refactoring. -Neil
В списке pgsql-patches по дате отправления: