On Mon, Oct 21, 2013 at 3:35 PM, pajato1 <notifications@github.com> wrote:

> In
> AndroidNativePlayerSampleApp/src/main/java/com/brightcove/android/sampleapp/MainActivity.java:
>
> > +    private void setupCuePoints() {
> > +        String cuePointType = "ad";
> > +        Map<String, Object> properties = new HashMap<String, Object>();
> > +        Map<String, Object> details = new HashMap<String, Object>();
> > +
> > +        // preroll
> > +        CuePoint cuePoint = new CuePoint(CuePoint.PositionType.BEFORE, cuePointType, properties);
> > +        details.put(Event.CUE_POINT, cuePoint);
> > +        eventEmitter.emit(EventType.SET_CUE_POINT, details);
> > +
> > +        // midroll
> > +        /*
> > +        cuePoint = new CuePoint(10, cuePointType, properties);
> > +        details.put(Event.CUE_POINT, cuePoint);
> > +        eventEmitter.emit(EventType.SET_CUE_POINT, details);
> > +        */
>
> Midrolls don't work with HLS due some bugs in the Android MediaPlayer, so
> it makes sense to leave this commented out, but a comment saying why it's
> commented out seems like a good idea.
>

A comment has been added to the commented out code.

-dad