Reviewed-by: Ian Romanick <[email protected]> On 09/29/2016 11:37 PM, Timothy Arceri wrote: > The warning was also the wrong location, it should have been > in the else. > --- > src/mesa/drivers/dri/i915/intel_fbo.h | 4 +--- > src/mesa/drivers/dri/i965/intel_fbo.h | 4 +--- > 2 files changed, 2 insertions(+), 6 deletions(-) > > diff --git a/src/mesa/drivers/dri/i915/intel_fbo.h > b/src/mesa/drivers/dri/i915/intel_fbo.h > index 114dd68..769dab8 100644 > --- a/src/mesa/drivers/dri/i915/intel_fbo.h > +++ b/src/mesa/drivers/dri/i915/intel_fbo.h > @@ -89,10 +89,8 @@ static inline struct intel_renderbuffer * > intel_renderbuffer(struct gl_renderbuffer *rb) > { > struct intel_renderbuffer *irb = (struct intel_renderbuffer *) rb; > - if (irb && irb->Base.Base.ClassID == INTEL_RB_CLASS) { > - /*_mesa_warning(NULL, "Returning non-intel Rb\n");*/ > + if (irb && irb->Base.Base.ClassID == INTEL_RB_CLASS) > return irb; > - } > else > return NULL; > } > diff --git a/src/mesa/drivers/dri/i965/intel_fbo.h > b/src/mesa/drivers/dri/i965/intel_fbo.h > index 89894cd..e6f6156 100644 > --- a/src/mesa/drivers/dri/i965/intel_fbo.h > +++ b/src/mesa/drivers/dri/i965/intel_fbo.h > @@ -130,10 +130,8 @@ static inline struct intel_renderbuffer * > intel_renderbuffer(struct gl_renderbuffer *rb) > { > struct intel_renderbuffer *irb = (struct intel_renderbuffer *) rb; > - if (irb && irb->Base.Base.ClassID == INTEL_RB_CLASS) { > - /*_mesa_warning(NULL, "Returning non-intel Rb\n");*/ > + if (irb && irb->Base.Base.ClassID == INTEL_RB_CLASS) > return irb; > - } > else > return NULL; > } >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
