On Wed, 3 May 2017, Alexandra Hájková wrote:

---
libavcodec/arm/hevc_idct.S | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

The change itself is ok, but the commit message is misleading. While this is more correct than before, it doesn't actually change the result of the code, so it did execute correctly even before.

Perhaps:

"hevc: ... foo bar: Use the right element size for stores

This improves readability but doesn't change the actual behaviour of the code."

Or something similar. Or see 3dd7827258ddaa2e51085d0c677d6f3b1be3572f as an example of a similar commit message.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to