On 4/22/2017 2:27 AM, wm4 wrote:
On Sat, 22 Apr 2017 16:04:22 +0700
Muhammad Faiz <mfc...@gmail.com> wrote:

Signed-off-by: Muhammad Faiz <mfc...@gmail.com>
---

+1 to patches 1-7. As long as the accessors only trivially access
public fields, it's better (and less ugly) not to use accessors at all.

Then why have the accessors at all if the fields are public? As far as I can tell, the benefit to using these accessors is that, if certain properties become internal in the future, or variable names change, or the structures become opaque, etc, the code will continue to work.

Aaron Levinson
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to