diff options
| author |   Kristaps Dzonsons <kristaps@bsd.lv> | 2011-04-09 15:29:40 +0000 |
|---|---|---|
| committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-04-09 15:29:40 +0000 |
| commit | dc6820cf87c36506f7d235493e75c1520231e9d9 (patch) | |
| tree | e5c60975fd48bf07a47693f981a0ff9991db03b9 /read.c | |
| parent | 0383763dbd62d52405be4dc65ae9aaff035c8f79 (diff) | |
| download | mandoc-dc6820cf87c36506f7d235493e75c1520231e9d9.tar.gz mandoc-dc6820cf87c36506f7d235493e75c1520231e9d9.tar.zst mandoc-dc6820cf87c36506f7d235493e75c1520231e9d9.zip | |
Remove a2roffdeco() and mandoc_special() functions and replace them with
a public (mandoc.h) function mandoc_escape(), which merges the
functionality of both prior functions.
Reason: code duplication. The a2roffdeco() and mandoc_special()
functions were pretty much the same thing and both quite complex. This
allows one function to receive improvements in (e.g.) subexpression
handling and performance, instead of having to replicate functionality.
As such, the mandoc_escape() function already handles a superset of the
escapes handled in previous versions and has improvements in performance
(using strcspn(), for example) and reliable handling of subexpressions.
This code Works For Me, but may need work to catch any regressions.
Since the benefits are great (leaner code, simpler API), I'd rather have
it in-tree than floating as a patch.
Diffstat (limited to 'read.c')
| -rw-r--r-- | read.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -1,4 +1,4 @@ -/* $Id: read.c,v 1.11 2011/04/04 23:04:38 kristaps Exp $ */ +/* $Id: read.c,v 1.12 2011/04/09 15:29:40 kristaps Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010, 2011 Ingo Schwarze <schwarze@openbsd.org> @@ -142,7 +142,7 @@ static const char * const mandocerrs[MANDOCERR_MAX] = { "tab in non-literal context", "end of line whitespace", "bad comment style", - "unknown escape sequence", + "bad escape sequence", "unterminated quoted string", "generic error", |