On 4/22/2017 2:27 AM, wm4 wrote:
On Sat, 22 Apr 2017 16:04:22 +0700
Muhammad Faiz <[email protected]> wrote:
Signed-off-by: Muhammad Faiz <[email protected]>
---
+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
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
