metsci / glimpse

Dynamic and Interactive Visualization of Big Data In Java and OpenGL
http://glimpse.metsci.com
Other
17 stars 6 forks source link

GlimpseDataPainter2D should not hide GlimpseContext #15

Open ulmangt opened 12 years ago

ulmangt commented 12 years ago

GlimpseDataPainter2D.paintTo() passes a GL handle but not the entire GlimpseContext. This makes it impossible to call the paintTo() method of other delegate painters which are not GlimpseDataPainter2D.

Suggest potentially moving the bounds.isValid() check up to GlimpsePainter2D, removing the axis null checks (which are redundant because GlimpsePainter2D makes those checks), and having users override paintTo() and call super.paintTo().