search

FFmpeg / FFV1

The FFV1 lossless video codec specification.
Other
154 stars 35 forks source link

Ballot Robert #247

Closed JeromeMartinez closed 3 years ago

JeromeMartinez commented 3 years ago

Address some issues from Robert Wilton's ballot. I don't think that I will handle other comments as I don't think we could do better, I answer to Robert about that.

JeromeMartinez commented 3 years ago

PR rebased after merge of other PRs.

JeromeMartinez commented 3 years ago

svg name issues fixed, comma added, @dericed please review again.

JeromeMartinez commented 3 years ago

I added the missing spaces during the move of the section. I changed the ESC commit to a move, from comments it looks like that ESC is not enough used for being worth to be at the beginning in the definitions. @dericed please review again.

dericed commented 3 years ago

thx, reviewed and tested, thanks for addressing all points.

michaelni commented 3 years ago

d1d3da1efb89efc96f5207cbf4ffcd37cb10aa06 adds a line thats removed in the next commit

JeromeMartinez commented 3 years ago

d1d3da1 adds a line thats removed in the next commit

Fixed, please review again / merge.