[Android] [PATCH 06/13] libvlc: add VLCObject

Thomas Guillem thomas at gllm.fr
Thu Jan 15 21:07:34 CET 2015


On Thu, Jan 15, 2015, at 18:27, Edward Wang wrote:
> On 2015-01-15 10:22, Thomas Guillem <thomas at gllm.fr> wrote:
> > +package org.videolan.libvlc;
> > +
> > +public abstract class VLCObject {
> > +    private final static String TAG = "LibVLC/VlcObject";
> > +
> > +    public static class Events {
> 
> Why a nested class for this? It might be nicer to put Events into 
> org.videolan.libvlc directly rather than have it nested inside VLCObject.

Because these Events are specific to VLCObject and all inherited
classes.
>From my point of view, it's cleaner that way

> 
> Regards,
>          Edward Wang
> _______________________________________________
> Android mailing list
> Android at videolan.org
> https://mailman.videolan.org/listinfo/android


More information about the Android mailing list