< prev index next >
src/solaris/classes/sun/awt/X11/XScrollbar.java
Print this page
rev 1571 : 8010297: Missing isLoggable() checks in logging code
Summary: Add isLoggable() checks
Reviewed-by: anthony, mchung, serb
Contributed-by: Laurent Bourges <bourges.laurent@gmail.com>
*** 116,126 ****
}
abstract protected void rebuildArrows();
public void setSize(int width, int height) {
! if (log.isLoggable(Level.FINER)) log.finer("Setting scroll bar " + this + " size to " + width + "x" + height);
this.width = width;
this.height = height;
}
/**
--- 116,128 ----
}
abstract protected void rebuildArrows();
public void setSize(int width, int height) {
! if (log.isLoggable(Level.FINER)) {
! log.finer("Setting scroll bar " + this + " size to " + width + "x" + height);
! }
this.width = width;
this.height = height;
}
/**
*** 162,172 ****
* @param width the width of the scrollbar
* @param height the height of the scrollbar
* @param paintAll paint the whole scrollbar if true, just the thumb is false
*/
void paint(Graphics g, Color colors[], boolean paintAll) {
! if (log.isLoggable(Level.FINER)) log.finer("Painting scrollbar " + this);
boolean useBufferedImage = false;
Graphics2D g2 = null;
BufferedImage buffer = null;
if (!(g instanceof Graphics2D)) {
--- 164,176 ----
* @param width the width of the scrollbar
* @param height the height of the scrollbar
* @param paintAll paint the whole scrollbar if true, just the thumb is false
*/
void paint(Graphics g, Color colors[], boolean paintAll) {
! if (log.isLoggable(Level.FINER)) {
! log.finer("Painting scrollbar " + this);
! }
boolean useBufferedImage = false;
Graphics2D g2 = null;
BufferedImage buffer = null;
if (!(g instanceof Graphics2D)) {
*** 333,343 ****
--- 337,349 ----
/**
* Tell the scroller to start scrolling.
*/
void startScrolling() {
+ if (log.isLoggable(Level.FINER)) {
log.finer("Start scrolling on " + this);
+ }
// Make sure that we scroll at least once
scroll();
// wake up the scroll repeater
if (scroller == null) {
*** 353,363 ****
--- 359,371 ----
/**
* Tell the instance scroller to start scrolling.
* See 6243382 for more information
*/
void startScrollingInstance() {
+ if (log.isLoggable(Level.FINER)) {
log.finer("Start scrolling on " + this);
+ }
// Make sure that we scroll at least once
scroll();
i_scroller.setScrollbar(this);
i_scroller.start();
*** 366,376 ****
--- 374,386 ----
/**
* Tell the instance scroller to stop scrolling.
* See 6243382 for more information
*/
void stopScrollingInstance() {
+ if (log.isLoggable(Level.FINER)) {
log.finer("Stop scrolling on " + this);
+ }
i_scroller.stop();
}
/**
< prev index next >