diff options
author | Wei Yang <richardw.yang@linux.intel.com> | 2019-06-10 11:08:51 +0800 |
---|---|---|
committer | Juan Quintela <quintela@redhat.com> | 2019-07-15 15:39:02 +0200 |
commit | 7c960d617ad4fee940c5bb4df96e91ae31f55d18 (patch) | |
tree | 0761d453f8f46442b48fc73ae4ff83f1a7df254b /util/cutils.c | |
parent | ca3538039012258973ab75775109dbe9577097f6 (diff) | |
download | qemu-7c960d617ad4fee940c5bb4df96e91ae31f55d18.zip qemu-7c960d617ad4fee940c5bb4df96e91ae31f55d18.tar.gz qemu-7c960d617ad4fee940c5bb4df96e91ae31f55d18.tar.bz2 |
cutils: remove one unnecessary pointer operation
Since we will not operate on the next address pointed by out, it is not
necessary to do addition on it.
After removing the operation, the function size reduced 16/18 bytes.
Signed-off-by: Wei Yang <richardw.yang@linux.intel.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20190610030852.16039-2-richardw.yang@linux.intel.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'util/cutils.c')
-rw-r--r-- | util/cutils.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/util/cutils.c b/util/cutils.c index dfc605f..fd591ca 100644 --- a/util/cutils.c +++ b/util/cutils.c @@ -756,11 +756,11 @@ int uleb128_encode_small(uint8_t *out, uint32_t n) { g_assert(n <= 0x3fff); if (n < 0x80) { - *out++ = n; + *out = n; return 1; } else { *out++ = (n & 0x7f) | 0x80; - *out++ = n >> 7; + *out = n >> 7; return 2; } } @@ -768,7 +768,7 @@ int uleb128_encode_small(uint8_t *out, uint32_t n) int uleb128_decode_small(const uint8_t *in, uint32_t *n) { if (!(*in & 0x80)) { - *n = *in++; + *n = *in; return 1; } else { *n = *in++ & 0x7f; @@ -776,7 +776,7 @@ int uleb128_decode_small(const uint8_t *in, uint32_t *n) if (*in & 0x80) { return -1; } - *n |= *in++ << 7; + *n |= *in << 7; return 2; } } |