Start line:  
End line:  

Snippet Preview

Snippet HTML Code

Stack Overflow Questions
  /*
   * fb-contrib - Auxiliary detectors for Java programs
   * Copyright (C) 2005-2013 Dave Brosius
   *
   * This library is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser General Public
  * License along with this library; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  */
 package com.mebigfatguy.fbcontrib.detect;
 
 import java.util.Map;
 import java.util.Set;
 
 import  org.apache.bcel.classfile.Code;
 
 
 import  edu.umd.cs.findbugs.BugInstance;
 import  edu.umd.cs.findbugs.BugReporter;
 import  edu.umd.cs.findbugs.BytecodeScanningDetector;
 import  edu.umd.cs.findbugs.OpcodeStack;
 import  edu.umd.cs.findbugs.ba.ClassContext;

looks for creation of java.awt.Graphics object that do not have the .dispose() method called on them when finished. These objects will be cleaned up by the Garbage collector, bug given the likelyhood that large numbers of these objects can be created in a short period of time, it is better to dispose them as soon as possible
 
 public class LingeringGraphicsObjects extends BytecodeScanningDetector {
 
 	private static final Set<StringGRAPHICS_PRODUCERS = new HashSet<String>();
 	static {
 		.add("java/awt/image/BufferedImage#getGraphics()Ljava/awt/Graphics;");
 		.add("java/awt/Graphics#create()Ljava/awt/Graphics;");
 	}
 
 	private final BugReporter bugReporter;
 	private OpcodeStack stack;
 	private Map<IntegerIntegergraphicsRegs// reg->pc
 
 	public LingeringGraphicsObjects(BugReporter bugReporter) {
 		this. = bugReporter;
 	}
 
 	public void visitClassContext(ClassContext classContext) {
 		try {
 			 = new OpcodeStack();
 			super.visitClassContext(classContext);
 		} finally {
 			 = null;
 			 = null;
 		}
 	}
 
 	public void visitCode(Code obj) {
 		.resetForMethodEntry(this);
 		super.visitCode(obj);
 		for (Integer pc : .values()) {
 			.reportBug(new BugInstance(this"LGO_LINGERING_GRAPHICS_OBJECT", NORMAL_PRIORITY)
 					.addClass(this).addMethod(this).addSourceLine(thispc.intValue()));
 		}
 
 	}
 
 	public void sawOpcode(int seen) {
 		Integer sawNewGraphicsAt = null;
 		try {
 			switch (seen) {
 				case ALOAD:
 				case ALOAD_0:
 				case ALOAD_1:
 				case ALOAD_2:
 				case ALOAD_3: {
 					int reg = RegisterUtils.getALoadReg(thisseen);
 					sawNewGraphicsAt = .get(Integer.valueOf(reg));
 				}
 				break;
 
 				case ASTORE:
				case ASTORE_0:
				case ASTORE_1:
				case ASTORE_2:
				case ASTORE_3: {
					if (.getStackDepth() > 0) {
						OpcodeStack.Item item = .getStackItem(0);
						sawNewGraphicsAt = (Integeritem.getUserValue();
						Integer reg = Integer.valueOf(RegisterUtils.getAStoreReg(thisseen));
						if (sawNewGraphicsAt != null) {
							.put(regsawNewGraphicsAt);
else {
						}
						sawNewGraphicsAt = null;
					}
				}
				break;
				case ARETURN:
					if (.getStackDepth() > 0) {
						OpcodeStack.Item item = .getStackItem(0);
						.remove(Integer.valueOf(item.getRegisterNumber()));
					}
				break;
				case INVOKEVIRTUAL:
					String methodName = getNameConstantOperand();
					String methodSig = getSigConstantOperand();
					String methodInfo = clsName + "#" + methodName + methodSig;
					if (.contains(methodInfo)) {
						sawNewGraphicsAt = Integer.valueOf(getPC());
else {
						if (("java/awt/Graphics#dispose()V".equals(methodInfo))
								|| ("java/awt/Graphics2D#dispose()V".equals(methodInfo))) {
							if (.getStackDepth() > 0) {
								OpcodeStack.Item item = .getStackItem(0);
								.remove(Integer.valueOf(item.getRegisterNumber()));
							}
						}
					}
				break;
			}
finally {
			TernaryPatcher.pre(seen);
			.sawOpcode(thisseen);
			TernaryPatcher.post(seen);
			if (sawNewGraphicsAt != null) {
				if (.getStackDepth() > 0) {
					OpcodeStack.Item item = .getStackItem(0);
					item.setUserValue(sawNewGraphicsAt);
				}
			}
		}
	}
New to GrepCode? Check out our FAQ X