Re: [HACKERS] Minor bugs and a formatting gripe
От | Bruce Momjian |
---|---|
Тема | Re: [HACKERS] Minor bugs and a formatting gripe |
Дата | |
Msg-id | 199809040502.BAA01732@candle.pha.pa.us обсуждение исходный текст |
Ответ на | Minor bugs and a formatting gripe (Tom Lane <tgl@sss.pgh.pa.us>) |
Список | pgsql-hackers |
> Bruce mentioned that he'd had some trouble applying my last set of > patches, because he'd already run his reformatting script against > the files involved. So, I took the trouble to grovel through a > "diff --ignore-all-space" comparison of tonight's cvs tree with what > I had last week. (I didn't have the energy to go through the *whole* > tree, but I did look through all of src/interfaces/.) It was a > worthwhile exercise because I found a couple of minor errors --- see > attached patch. > > I was mostly pretty happy with what the formatting script had done with > the code, especially in libpgtcl which had not been anywhere close to > matching the formatting conventions of the rest of pgsql. Nice job! > > BUT: I am not happy that the script chooses to reflow block comments. > Line breaks chosen on the basis of how much fits are not a readability > improvement over line breaks placed where a human author felt they made > sense. For example, is this really an improvement in readability? > (And if it's *not* a clear gain, why are we doing it?) Thanks for checking on my patch job. Patches applied. -- Bruce Momjian | 830 Blythe Avenue maillist@candle.pha.pa.us | Drexel Hill, Pennsylvania 19026 + If your life is a hard drive, | (610) 353-9879(w) + Christ can be your backup. | (610) 853-3000(h)
В списке pgsql-hackers по дате отправления: