]> git.cameronkatri.com Git - mandoc.git/commitdiff
Fix a minor optimization i broke in rev. 1.163 on August 20, 2010:
authorIngo Schwarze <schwarze@openbsd.org>
Fri, 25 Apr 2014 14:11:30 +0000 (14:11 +0000)
committerIngo Schwarze <schwarze@openbsd.org>
Fri, 25 Apr 2014 14:11:30 +0000 (14:11 +0000)
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!

mdoc.c

diff --git a/mdoc.c b/mdoc.c
index 168778177c42dca0cb4f2daf2795ac245debcc29..c56fad41a4a8ab96cddf09562c135a0f132ecd01 100644 (file)
--- a/mdoc.c
+++ b/mdoc.c
@@ -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>
 /*
  * 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';
 
 
        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,
 
        if (MDOC_MAX == tok) {
                mandoc_vmsg(MANDOCERR_MACRO, mdoc->parse,