]> granicus.if.org Git - neomutt/commitdiff
reorder params in driver_tags_get_transformed_for()
authorRichard Russon <rich@flatcap.org>
Tue, 20 Aug 2019 17:30:55 +0000 (18:30 +0100)
committerRichard Russon <rich@flatcap.org>
Wed, 21 Aug 2019 13:12:06 +0000 (14:12 +0100)
Change the parameter order to match all the other tags functions.

email/tags.c
email/tags.h
hdrline.c
test/tags/driver_tags_get_transformed_for.c

index b6c5989ea922712d95d12245153b5cac1ccbcd20..c6ac5d4135bbbc7c7f52407a4dc1ad92e06ee8cd 100644 (file)
@@ -158,14 +158,14 @@ char *driver_tags_get_with_hidden(struct TagHead *head)
 
 /**
  * driver_tags_get_transformed_for - Get transformed tag for a tag name from a header
- * @param[in] name Tag to transform
  * @param[in] head List of tags
+ * @param[in] name Tag to transform
  * @retval ptr String tag
  *
  * Return a new allocated string containing all tags separated by space even
  * the hiddens.
  */
-char *driver_tags_get_transformed_for(const char *name, struct TagHead *head)
+char *driver_tags_get_transformed_for(struct TagHead *head, const char *name)
 {
   return driver_tags_getter(head, true, true, name);
 }
index 1c5481751c86de63992a33961dc43c0d1f159baa..1c7973ee5443632a4a54c1f8655afd4cc7b486b1 100644 (file)
@@ -49,7 +49,7 @@ STAILQ_HEAD(TagHead, TagNode);
 void  driver_tags_free               (struct TagHead *head);
 char *driver_tags_get                (struct TagHead *head);
 char *driver_tags_get_transformed    (struct TagHead *head);
-char *driver_tags_get_transformed_for(const char *name, struct TagHead *head);
+char *driver_tags_get_transformed_for(struct TagHead *head, const char *name);
 char *driver_tags_get_with_hidden    (struct TagHead *head);
 bool  driver_tags_replace            (struct TagHead *head, char *tags);
 
index c5a61c0ac78c899e0c7cf939fb581fad53873321..a2f1ba09d39656689abc60159168b5110c107560 100644 (file)
--- a/hdrline.c
+++ b/hdrline.c
@@ -926,7 +926,7 @@ static const char *index_format_str(char *buf, size_t buflen, size_t col, int co
         tag = mutt_hash_find(TagFormats, format);
         if (tag)
         {
-          tags = driver_tags_get_transformed_for(tag, &e->tags);
+          tags = driver_tags_get_transformed_for(&e->tags, tag);
           colorlen = add_index_color(buf, buflen, flags, MT_COLOR_INDEX_TAG);
           mutt_format_s(buf + colorlen, buflen - colorlen, prec, NONULL(tags));
           add_index_color(buf + colorlen, buflen - colorlen, flags, MT_COLOR_INDEX);
@@ -943,7 +943,7 @@ static const char *index_format_str(char *buf, size_t buflen, size_t col, int co
         tag = mutt_hash_find(TagFormats, format);
         if (tag)
         {
-          tags = driver_tags_get_transformed_for(tag, &e->tags);
+          tags = driver_tags_get_transformed_for(&e->tags, tag);
           if (!tags)
             optional = 0;
           FREE(&tags);
index 5608aaa46c3d4e377dd597d688f0f139453e5b79..f8e03fd7af44bf3848452009167372ba449bd8ec 100644 (file)
 
 void test_driver_tags_get_transformed_for(void)
 {
-  // char *driver_tags_get_transformed_for(const char *name, struct TagHead *head);
+  // char *driver_tags_get_transformed_for(struct TagHead *head, const char *name);
 
   {
     struct TagHead taghead = { 0 };
-    TEST_CHECK(!driver_tags_get_transformed_for(NULL, &taghead));
+    TEST_CHECK(!driver_tags_get_transformed_for(&taghead, NULL));
   }
 
   {
-    TEST_CHECK(!driver_tags_get_transformed_for("apple", NULL));
+    TEST_CHECK(!driver_tags_get_transformed_for(NULL, "apple"));
   }
 }