Remove need for padding in av_memcpy_backptr()
authorMans Rullgard <mans@mansr.com>
Tue, 23 Oct 2012 17:05:45 +0000 (18:05 +0100)
committerMans Rullgard <mans@mansr.com>
Tue, 23 Oct 2012 17:57:24 +0000 (18:57 +0100)
Signed-off-by: Mans Rullgard <mans@mansr.com>
libavutil/lzo.c
libavutil/lzo.h

index 101d4f462e25180d93bb6f0a52caafb17c5046f2..c17d32f3625dc488d037073d6e28f03eec58040e 100644 (file)
@@ -132,13 +132,14 @@ static inline void memcpy_backptr(uint8_t *dst, int back, int cnt)
     if (back == 1) {
         memset(dst, *src, cnt);
     } else {
-#ifdef OUTBUF_PADDED
-        AV_COPY16U(dst,     src);
-        AV_COPY16U(dst + 2, src + 2);
-        src += 4;
-        dst += 4;
-        cnt -= 4;
-        if (cnt > 0) {
+        if (cnt >= 4) {
+            AV_COPY16U(dst,     src);
+            AV_COPY16U(dst + 2, src + 2);
+            src += 4;
+            dst += 4;
+            cnt -= 4;
+        }
+        if (cnt >= 8) {
             AV_COPY16U(dst,     src);
             AV_COPY16U(dst + 2, src + 2);
             AV_COPY16U(dst + 4, src + 4);
@@ -147,7 +148,6 @@ static inline void memcpy_backptr(uint8_t *dst, int back, int cnt)
             dst += 8;
             cnt -= 8;
         }
-#endif
         if (cnt > 0) {
             int blocklen = back;
             while (cnt > blocklen) {
index e3c6c01c6be8c94b6594110df421560302d29a69..a84b9bd15db12dd154bce56e59815e2637d91d67 100644 (file)
@@ -61,7 +61,7 @@ int av_lzo1x_decode(void *out, int *outlen, const void *in, int *inlen);
 
 /**
  * @brief deliberately overlapping memcpy implementation
- * @param dst destination buffer; must be padded with 12 additional bytes
+ * @param dst destination buffer
  * @param back how many bytes back we start (the initial size of the overlapping window)
  * @param cnt number of bytes to copy, must be >= 0
  *