Agree on all review items. Will implement all requested changes.
On 23/07/2013 1:51 PM, "Cory Fields" <notifications@github.com> wrote:

> In xbmc/windowing/egl/EGLNativeTypeWayland.cpp:
>
> > + */
> > +#include <sstream>
> > +#include <iostream>
> > +#include <stdexcept>
> > +
> > +#include <boost/noncopyable.hpp>
> > +#include <boost/function.hpp>
> > +#include <boost/bind.hpp>
> > +#include <boost/scoped_ptr.hpp>
> > +#include <boost/shared_ptr.hpp>
> > +
> > +#include <cstdlib>
> > +
> > +#include <wayland-client.h>
> > +
> > +#include "../DllWaylandClient.h"
>
> please avoid including parent dirs. xbmc/ is always in the includes list,
> you should list relative to that.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/xbmc/xbmc/pull/2978/files#r5356609>
> .
>