diff options
| author | Kaz Kylheku <kaz@kylheku.com> | 2012-02-26 20:58:31 -0800 |
|---|---|---|
| committer | Kaz Kylheku <kaz@kylheku.com> | 2012-02-26 20:58:31 -0800 |
| commit | 73625e0ae9b4fde9c6ff64bd6dc22a1b7797be1a (patch) | |
| tree | 44308c28567baad646762322f84bb501262ada66 /ChangeLog | |
| parent | 7649d799041454321809da89a5716afc19c34f3d (diff) | |
| download | txr-73625e0ae9b4fde9c6ff64bd6dc22a1b7797be1a.tar.gz txr-73625e0ae9b4fde9c6ff64bd6dc22a1b7797be1a.tar.bz2 txr-73625e0ae9b4fde9c6ff64bd6dc22a1b7797be1a.zip | |
Fixing long-time (pre-GIT) bug. The object (nil) was stupidly used to
represent empty optional output clauses, distinguishing them from
missing clauses. This creates an ambiguity, so that an @(output)
block which puts out a single empty line is treated as empty.
Present but empty clauses are now represented by t.
* match.c (do_output_line): Check for t and bail.
(do_output): Check for t instead of (nil) and bail.
* parser.y (o_elems_opt2): Nonterminal deleted.
(out_clauses_opt): Empty case generates nil.
(req_parts_opt): o_elems_opt2 replaced by o_elems_opt.
(repeat_rep_helper): Function now keeps track of which
clauses were specified. For those that were specified, but
empty, it substitutes t.
* tests/008/empty-clauses.expected: New file.
* tests/008/empty-clauses.txr: New file.
Diffstat (limited to 'ChangeLog')
| -rw-r--r-- | ChangeLog | 22 |
1 files changed, 22 insertions, 0 deletions
@@ -1,5 +1,27 @@ 2012-02-26 Kaz Kylheku <kaz@kylheku.com> + Fixing long-time (pre-GIT) bug. The object (nil) was stupidly used to + represent empty optional output clauses, distinguishing them from + missing clauses. This creates an ambiguity, so that an @(output) + block which puts out a single empty line is treated as empty. + Present but empty clauses are now represented by t. + + * match.c (do_output_line): Check for t and bail. + (do_output): Check for t instead of (nil) and bail. + + * parser.y (o_elems_opt2): Nonterminal deleted. + (out_clauses_opt): Empty case generates nil. + (req_parts_opt): o_elems_opt2 replaced by o_elems_opt. + (repeat_rep_helper): Function now keeps track of which + clauses were specified. For those that were specified, but + empty, it substitutes t. + + * tests/008/empty-clauses.expected: New file. + + * tests/008/empty-clauses.txr: New file. + +2012-02-26 Kaz Kylheku <kaz@kylheku.com> + Bug #35625 * parser.l (BSCHR, BSYM, BTOK): New lexical definitions. |