Code review comment for lp:~goelkunal/kdelibs/debian

Revision history for this message
Steve Langasek (vorlon) wrote :

The patch itself seems to be straightforward, but why are the glClearColor(), glClear() calls not needed here? Presumably they were added for a reason, and we should be sure that reason no longer applies before making such a change.

I see that these API calls are also part of GLES; so if these calls need to be kept, an alternative would be for plasma to select either GL or GLES as needed.

review: Needs Information

« Back to merge proposal