Re: [HACKERS] Running pgindent
От | Bruce Momjian |
---|---|
Тема | Re: [HACKERS] Running pgindent |
Дата | |
Msg-id | 199802191559.KAA24718@candle.pha.pa.us обсуждение исходный текст |
Ответ на | Re: [HACKERS] Running pgindent ("Thomas G. Lockhart" <lockhart@alumni.caltech.edu>) |
Список | pgsql-hackers |
> > Bruce Momjian wrote: > > > I want to run pgindent before the final release. Who has outstanding > > patches they are sitting on that would be affected by this? Only places > > where we have added non-conforming code would be affected. > > Ack! Can you make at least three changes to pgindent before doing this? > > 1) for functions which look like > datetime * > abstime_datetime(int4 x) > { > ... > do not put a tab (or extra spaces) between "datetime" and "*". > > For the end of functions which look like > ... > } /* abstime_datetime() */ > > do not put any tabs (or just one) between the "{" and the comment. > > Also, some comments get wrapped to the next line, making some ugly sources; > perhaps pgindent should not touch comment lines at all? Or at least keep > things on the same line, messing with the tabbing only?? Well, that is what indent does. I can try and work around it, but I am afraid I will mess it up somehow. Right now, all the stuff is indented consistently, and running it again will only make the changed stuff look like the rest. Making a change will change all the code, even the unchanged stuff. Also, for comments, it will not change comments that are block-style. See the indent manual page and the flags I use for help. Keep in mind I use BSD indent, which does not have the bugs in GNU indent. -- Bruce Momjian maillist@candle.pha.pa.us
В списке pgsql-hackers по дате отправления: