diff options
author | Max Kellermann <max@duempel.org> | 2008-08-28 20:02:16 +0200 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2008-08-28 20:02:16 +0200 |
commit | 801c71ed1c02a1505ef64902f6557fdfe1749a96 (patch) | |
tree | 3a9b9d038a6a08938d1518f93434468f0e1408a1 /src/tag.c | |
parent | 1560a70b01e1883d2cf6d444cef26f28753bb94e (diff) | |
download | mpd-801c71ed1c02a1505ef64902f6557fdfe1749a96.tar.gz mpd-801c71ed1c02a1505ef64902f6557fdfe1749a96.tar.xz mpd-801c71ed1c02a1505ef64902f6557fdfe1749a96.zip |
unsigned integers and size_t
Use "unsigned int" whenever negative values are not meaningful. Use
size_t whenever we are going to describe buffer sizes.
Diffstat (limited to 'src/tag.c')
-rw-r--r-- | src/tag.c | 7 |
1 files changed, 3 insertions, 4 deletions
@@ -25,7 +25,6 @@ #include "charConv.h" #include "tagTracker.h" #include "song.h" -#include "os_compat.h" #ifdef HAVE_ID3TAG # define isId3v1(tag) (id3_tag_options(tag, 0, 0) & ID3_TAG_OPTION_ID3V1) @@ -698,9 +697,9 @@ int mpdTagsAreEqual(MpdTag * tag1, MpdTag * tag2) } static void appendToTagItems(MpdTag * tag, enum tag_type type, - char *value, int len) + char *value, size_t len) { - int i = tag->numOfItems; + unsigned int i = tag->numOfItems; char *duplicated = xmalloc(len + 1); memcpy(duplicated, value, len); @@ -719,7 +718,7 @@ static void appendToTagItems(MpdTag * tag, enum tag_type type, } void addItemToMpdTagWithLen(MpdTag * tag, enum tag_type itemType, - char *value, int len) + char *value, size_t len) { if (ignoreTagItems[itemType]) { |