Hi Joel, Hi Akim, All > Please try to follow the format you see in other ChangeLog entries. > Specifically, put top-level constructs in parentheses after the file > names, and don't add extra colons. > > Throughout Bison documentation (including the ChangeLog and source code > comments), we prefer two spaces after each period.
I pushed a small patch (attached) fixing some of minor issues described in your mail. Thank you for the explanations - your pedantry is awesome. It's installed on master/branch-2.5. I will write a separate mail about the main issue. > > Please try to follow the format you see in other ChangeLog entries. > Specifically, put top-level constructs in parentheses after the file > names, and don't add extra colons. >> diff --git a/NEWS b/NEWS > > Throughout Bison documentation (including the ChangeLog and source code > comments), we prefer two spaces after each period. -- Best regards, Alex Rozenman ([email protected]).
commit 5b1ff42379edbbf37b135d33aa5fb3c72da2dca1 Author: Alex Rozenman <[email protected]> Date: Sat Oct 3 18:29:14 2009 +0200 Add additional space after periods in NEWS. * NEWS (2.5): here. diff --git a/ChangeLog b/ChangeLog index 1b0bb87..35b71ee 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2009-10-03 Alex Rozenman <[email protected]> + + Add additional space after periods in NEWS. + * NEWS (2.5): here. + 2009-09-29 Joel E. Denny <[email protected]> Use the correct conversion specifier for size_t. @@ -22,16 +27,17 @@ 2009-09-19 Alex Rozenman <[email protected]> Keep sub-messages aligned. Fix strings for translation. - * src/location.h: (location_print): Add return value. - * src/location.c: (location_print): Return number of printed + * src/location.h (location_print): Add return value. + * src/location.c (location_print): Return number of printed characters. - * src/complain.h: Two new functions (complain_at_indent, - warn_at_indent). - * src/complain.cpp: Implement the alignment mechanism. Add new - static variable (indent_ptr). Use and update it (error_message, - complain_at_indent, warn_at_indent). - * src/scan-code.l: Fix strings for translations. Use new *_indent - functions (parse_ref, show_sub_messages). + * src/complain.h (complain_at_indent, warn_at_indent): Prototype + new functions. + * src/complain.cpp (indent_ptr): New static variable. + (error_message, complain_at_indent, warn_at_indent): Implement + the alignment mechanism. + * src/scan-code.l (parse_ref, show_sub_messages): Fix strings + for translations. Use new alignment mechanism. + * tests/named-ref.at: Adjust test-cases. * NEWS (2.5): Add an announcement about named references. 2009-09-17 Akim Demaille <[email protected]> diff --git a/NEWS b/NEWS index c80c40f..d585139 100644 --- a/NEWS +++ b/NEWS @@ -71,11 +71,11 @@ Bison News stmt[res] : 'if' expr[cond] 'then' stmt[then] 'else' stmt[else] ';' { $res = mk_if_stmt($cond, $then, $else); } - Location information is also accessible using @name syntax. When + Location information is also accessible using @name syntax. When accessing symbol names containing dots or dashes, explicit bracketing ($[sym.1]) must be used. - These features are experimental in this version. More user feedback + These features are experimental in this version. More user feedback will help to stabilize them. ** IELR(1) and Canonical LR(1) Support
