You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2017/11/11 17:26:02 UTC
svn commit: r1814962 -
/jmeter/trunk/src/core/org/apache/jmeter/gui/GuiPackage.java
Author: pmouawad
Date: Sat Nov 11 17:26:02 2017
New Revision: 1814962
URL: http://svn.apache.org/viewvc?rev=1814962&view=rev
Log:
Better logging
Modified:
jmeter/trunk/src/core/org/apache/jmeter/gui/GuiPackage.java
Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/GuiPackage.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/GuiPackage.java?rev=1814962&r1=1814961&r2=1814962&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/gui/GuiPackage.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/gui/GuiPackage.java Sat Nov 11 17:26:02 2017
@@ -259,7 +259,7 @@ public final class GuiPackage implements
comp = getGuiFromCache(guiClass, testClass);
nodesToGui.put(node, comp);
}
- log.debug("Gui retrieved = " + comp);
+ log.debug("Gui retrieved = {}", comp);
return comp;
} catch (Exception e) {
log.error("Problem retrieving gui", e);
@@ -298,7 +298,7 @@ public final class GuiPackage implements
currentNodeUpdated = false;
return comp;
} catch (Exception e) {
- log.error("Problem retrieving gui", e);
+ log.error("Problem retrieving current gui", e);
return null;
}
}
@@ -454,7 +454,7 @@ public final class GuiPackage implements
try {
if (currentNode != null && !currentNodeUpdated) {
- log.debug("Updating current node " + currentNode.getName());
+ log.debug("Updating current node {}", currentNode.getName());
JMeterGUIComponent comp = getGui(currentNode.getTestElement());
if (comp == null) {
log.debug("No component found for {}", currentNode.getName());
@@ -641,7 +641,7 @@ public final class GuiPackage implements
*/
public void displayPopUp(Component invoker, MouseEvent e, JPopupMenu popup) {
if (popup != null) {
- log.debug("Showing pop up for " + invoker + " at x,y = " + e.getX() + "," + e.getY());
+ log.debug("Showing pop up for {} at x,y = {},{}", invoker, e.getX(), e.getY());
popup.pack();
popup.show(invoker, e.getX(), e.getY());