Comments cleanup: - change the encoding of the original developer name from ISO-8859-1 to UTF-8 - remove the stale/completed TODO list - fix a typo
No code changes, only improved consistency in the comments. I kindly ask to apply this cleanup, which of course was due from the beginning. Attaching the patch. Regards, Rune
--- libavcodec/cinepakenc.c.orig 2016-12-05 23:07:54.000000000 +0100 +++ libavcodec/cinepakenc.c 2017-01-28 10:10:47.078999401 +0100 @@ -1,5 +1,5 @@ /* - * Cinepak encoder (c) 2011 Tomas H�rdin + * Cinepak encoder (c) 2011 Tomas Härdin * http://titan.codemill.se/~tomhar/cinepakenc.patch * * Fixes and improvements, vintage decoders compatibility @@ -23,9 +23,6 @@ ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. - * TODO: - * - optimize: color space conversion, ... - * - implement options to set the min/max number of strips? * MAYBE: * - "optimally" split the frame into several non-regular areas * using a separate codebook pair for each area and approximating @@ -92,7 +89,7 @@ #define MAX_STRIPS 32 //Note: having fewer choices regarding the number of strips speeds up encoding (obviously) #define MIN_STRIPS 1 //Note: having more strips speeds up encoding the frame (this is less obvious) // MAX_STRIPS limits the maximum quality you can reach -// when you want hight quality on high resolutions, +// when you want high quality on high resolutions, // MIN_STRIPS limits the minimum efficiently encodable bit rate // on low resolutions // the numbers are only used for brute force optimization for the first frame,
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel