diff options
author | Ingo Schwarze <schwarze@openbsd.org> | 2014-04-25 14:11:30 +0000 |
---|---|---|
committer | Ingo Schwarze <schwarze@openbsd.org> | 2014-04-25 14:11:30 +0000 |
commit | cb0cad5782ca3480821fec99f17bd057d5278ffe (patch) | |
tree | 0b116fe9beee6e4ae60869949a927acfa9cd2f18 | |
parent | 1ae7426d9fae79fc06cd0c737b91ababea935cd3 (diff) | |
download | mandoc-cb0cad5782ca3480821fec99f17bd057d5278ffe.tar.gz mandoc-cb0cad5782ca3480821fec99f17bd057d5278ffe.tar.zst mandoc-cb0cad5782ca3480821fec99f17bd057d5278ffe.zip |
Fix a minor optimization i broke in rev. 1.163 on August 20, 2010:
Do not bother looking into the hash table when the length of the macro
already tells us it's invalid. No functional change.
Noticed by jsg@, thanks!
-rw-r--r-- | mdoc.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -1,4 +1,4 @@ -/* $Id: mdoc.c,v 1.213 2014/04/20 16:46:05 schwarze Exp $ */ +/* $Id: mdoc.c,v 1.214 2014/04/25 14:11:30 schwarze Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010, 2012, 2013, 2014 Ingo Schwarze <schwarze@openbsd.org> @@ -865,7 +865,7 @@ mdoc_pmacro(struct mdoc *mdoc, int ln, char *buf, int offs) mac[i] = '\0'; - tok = (i > 1 || i < 4) ? mdoc_hash_find(mac) : MDOC_MAX; + tok = (i > 1 && i < 4) ? mdoc_hash_find(mac) : MDOC_MAX; if (MDOC_MAX == tok) { mandoc_vmsg(MANDOCERR_MACRO, mdoc->parse, |