diff options
author | wm4 <wm4@nowhere> | 2012-11-25 00:32:47 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2012-11-25 23:40:07 +0100 |
commit | 2bd7f980ac5b692b62b0765c4411129f953b0593 (patch) | |
tree | 400ce049da19e0a2afc80f3475460343d5514f53 /stream/http.c | |
parent | 99e178f1e8e2a6b9d7b3dd937f5214827f192c08 (diff) | |
download | mpv-2bd7f980ac5b692b62b0765c4411129f953b0593.tar.bz2 mpv-2bd7f980ac5b692b62b0765c4411129f953b0593.tar.xz |
memcpy_pic: kill useless/dangerous optimization
The memcpy_pic() function had a rather dangerous optimization: when the
limit2width flag was not set, it was allowed to overwrite the data
between the last pixel of a line and the first pixel of the next line
(i.e. write over the stride padding). That was also the reason why there
are so many whacky names for this function (memcpy_pic, my_memcpy_pic,
memcpy_pic2).
Kill this optimization, and never overwrite the stride padding. The code
doing this can still be used if there's no stride padding at all,
though.
Also use the name memcpy_pic for the proper function. Now it should be
rather clear that my_memcpy_pic and memcpy_pic2 are compatibility
aliases. They should go away over the time.
Diffstat (limited to 'stream/http.c')
0 files changed, 0 insertions, 0 deletions