Home/ Questions/Q 3300580
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-17T20:40:31+00:00

I am using a string as a lock and so want to ensure the

  • 0

I am using a string as a lock and so want to ensure the object is a new instance. FindBugs complains because it’s generally more efficient to define the string directly (with double quotes). My code looks like:

/** A lock for the list of inputs. */
@edu.umd.cs.findbugs.annotations.SuppressWarnings("DM_STRING_CTOR")
//We want a new String object here as this is a lock.
private final Object inputListLock = new String("inputListLock");

Am I doing something wrong here? The Eclipse FindBugs plugin is still reporting this as a problem:

Pattern id: DM_STRING_CTOR, type: Dm, category: PERFORMANCE

Using the java.lang.String(String) constructor wastes memory because the object so constructed will be functionally indistinguishable from the String passed as a parameter.  Just use the argument String directly.
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    Ok, so although both the other answers were interesting and useful (+1 for both), I didn’t end up changing the code and I’m going to accept my own answer. To satisfy FindBugs I moved the annotation from the member variable to the surrounding class.

    I’ve looked for some time but I haven’t found any information suggesting that the SuppressWarnings may only be applied to classes and methods. Neither have I found any examples of it being applied to member variables. So though this solution works I don’t know that it’s the ‘right’ solution (maybe there’s still something wrong with my FindBugs/Eclipse setup for example).

    • 0