From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from localhost (fantadrom.bsd.lv [local]) by fantadrom.bsd.lv (OpenSMTPD) with ESMTPA id ffb00fb2 for ; Tue, 23 Oct 2018 15:43:07 -0500 (EST) Date: Tue, 23 Oct 2018 15:43:07 -0500 (EST) X-Mailinglist: mandoc-source Reply-To: source@mandoc.bsd.lv MIME-Version: 1.0 From: schwarze@mandoc.bsd.lv To: source@mandoc.bsd.lv Subject: mandoc: The ctags(1) file format uses whitespace as a field delimiter, X-Mailer: activitymail 1.26, http://search.cpan.org/dist/activitymail/ Content-Type: text/plain; charset=utf-8 Message-Id: <381c6a72dcc7d67d@fantadrom.bsd.lv> Log Message: ----------- The ctags(1) file format uses whitespace as a field delimiter, and there is no escaping mechanism, so tags cannot contain whitespace. Consequently, we used to simply not tag macro arguments containing space characters. Instead, let's tag the first word, unless there is a proper match for that word somewhere else. For example, this makes ":tquery" work in ntpd.conf(5). Feature suggested by kn@, who also thinks the implementation looks reasonable and works in his testing. Modified Files: -------------- mandoc: tag.c Revision Data ------------- Index: tag.c =================================================================== RCS file: /home/cvs/mandoc/mandoc/tag.c,v retrieving revision 1.19 retrieving revision 1.20 diff -Ltag.c -Ltag.c -u -p -r1.19 -r1.20 --- tag.c +++ tag.c @@ -1,6 +1,6 @@ /* $Id$ */ /* - * Copyright (c) 2015, 2016 Ingo Schwarze + * Copyright (c) 2015, 2016, 2018 Ingo Schwarze * * Permission to use, copy, modify, and distribute this software for any * purpose with or without fee is hereby granted, provided that the above @@ -18,6 +18,7 @@ #include +#include #include #include #include @@ -121,41 +122,57 @@ fail: /* * Set the line number where a term is defined, - * unless it is already defined at a higher priority. + * unless it is already defined at a lower priority. */ void tag_put(const char *s, int prio, size_t line) { struct tag_entry *entry; + const char *se; size_t len; unsigned int slot; - /* Sanity checks. */ - if (tag_files.tfd <= 0) return; + if (s[0] == '\\' && (s[1] == '&' || s[1] == 'e')) s += 2; - if (*s == '\0' || strchr(s, ' ') != NULL) + + /* + * Skip whitespace and whatever follows it, + * and if there is any, downgrade the priority. + */ + + len = strcspn(s, " \t"); + if (len == 0) return; - slot = ohash_qlookup(&tag_data, s); + se = s + len; + if (*se != '\0') + prio = INT_MAX; + + slot = ohash_qlookupi(&tag_data, s, &se); entry = ohash_find(&tag_data, slot); if (entry == NULL) { /* Build a new entry. */ - len = strlen(s) + 1; - entry = mandoc_malloc(sizeof(*entry) + len); + entry = mandoc_malloc(sizeof(*entry) + len + 1); memcpy(entry->s, s, len); + entry->s[len] = '\0'; entry->lines = NULL; entry->maxlines = entry->nlines = 0; ohash_insert(&tag_data, slot, entry); } else { - /* Handle priority 0 entries. */ + /* + * Lower priority numbers take precedence, + * but 0 is special. + * A tag with priority 0 is only used + * if the tag occurs exactly once. + */ if (prio == 0) { if (entry->prio == 0) -- To unsubscribe send an email to source+unsubscribe@mandoc.bsd.lv