Revise from review

Author: idodeclare

src/org/opensolaris/opengrok/configuration/RuntimeEnvironment.java

@@ -1325,7 +1325,7 @@ public short getContextLimit() {
     public void setContextLimit(Short value)
             throws IllegalArgumentException {
         if (value < 1) {
-            throw new IllegalArgumentException("`value' is not positive");
+            throw new IllegalArgumentException("value is not positive");
         }
         contextLimit = value;
     }
@@ -1354,7 +1354,7 @@ public short getContextSurround() {
     public void setContextSurround(Short value)
             throws IllegalArgumentException {
         if (value < 0) {
-            throw new IllegalArgumentException("`value' is negative");
+            throw new IllegalArgumentException("value is negative");
         }
         contextSurround = value;
     }

src/org/opensolaris/opengrok/search/QueryBuilder.java

@@ -88,7 +88,7 @@ public class QueryBuilder {
      */
     public QueryBuilder reset(QueryBuilder other) {
         if (other == null) {
-            throw new IllegalArgumentException("`other' is null");
+            throw new IllegalArgumentException("other is null");
         }
         if (this != other) {
             queries.clear();

src/org/opensolaris/opengrok/search/context/Context.java

@@ -90,7 +90,7 @@ public class Context {
      */
     public Context(Query query, QueryBuilder qbuilder) {
         if (qbuilder == null) {
-            throw new IllegalArgumentException("`qbuilder' is null");
+            throw new IllegalArgumentException("qbuilder is null");
         }
 
         this.query = query;

src/org/opensolaris/opengrok/search/context/ContextArgs.java

@@ -50,11 +50,11 @@ public class ContextArgs {
     public ContextArgs(short contextSurround, short contextLimit) {
         if (contextSurround < 0) {
             throw new IllegalArgumentException(
-                "`contextSurround' cannot be negative");
+                "contextSurround cannot be negative");
         }
         if (contextLimit < 1) {
             throw new IllegalArgumentException(
-                "`contextLimit' must be positive");
+                "contextLimit must be positive");
         }
         this.contextSurround = contextSurround;
         this.contextLimit = contextLimit;

src/org/opensolaris/opengrok/search/context/ContextFormatter.java

@@ -149,7 +149,7 @@ public int getMoreLimit() {
      */
     public void setMoreLimit(int value) {
         if (value < 0) {
-            throw new IllegalArgumentException("`value' is negative");
+            throw new IllegalArgumentException("value is negative");
         }
         this.moreLimit = value;
     }
@@ -200,7 +200,7 @@ public void setScopes(Scopes value) {
     public Object format(Passage[] passages, String originalText) {
         String lineUrl = url;
         if (lineUrl == null) {
-            throw new IllegalStateException("`Url' property is null");
+            throw new IllegalStateException("Url property is null");
         }
 
         if (this.originalText == null || !this.originalText.equals(

src/org/opensolaris/opengrok/search/context/FormattedLines.java

@@ -92,7 +92,7 @@ public String pop() {
      */
     public String put(int lineno, String line) {
         if (line == null) {
-            throw new IllegalArgumentException("`line' is null");
+            throw new IllegalArgumentException("line is null");
         }
         return lines.put(lineno, line);
     }

src/org/opensolaris/opengrok/search/context/LineHighlight.java

@@ -25,7 +25,6 @@
 
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 import org.opensolaris.opengrok.web.HtmlConsts;
 import org.opensolaris.opengrok.web.Util;
@@ -48,7 +47,7 @@ public class LineHighlight {
 
     public LineHighlight(int lineno) {
         if (lineno < 0) {
-            throw new IllegalArgumentException("`lineno' cannot be negative");
+            throw new IllegalArgumentException("lineno cannot be negative");
         }
         this.lineno = lineno;
     }
@@ -98,7 +97,7 @@ public void condenseMarkups() {
      */
     public void addMarkup(PhraseHighlight phi) {
         if (phi == null) {
-            throw new IllegalArgumentException("`phi' is null");
+            throw new IllegalArgumentException("phi is null");
         }
         if (markups == null) {
             markups = new ArrayList<>();
@@ -126,7 +125,7 @@ public int getLelide() {
      */
     public void setLelide(int value) {
         if (value < 0) {
-            throw new IllegalArgumentException("`value' is negative");
+            throw new IllegalArgumentException("value is negative");
         }
         this.lelide = value;
     }
@@ -144,7 +143,7 @@ public int getRelide() {
      */
     public void setRelide(int value) {
         if (value < 0) {
-            throw new IllegalArgumentException("`value' is negative");
+            throw new IllegalArgumentException("value is negative");
         }
         this.relide = value;
     }

src/org/opensolaris/opengrok/search/context/OGKUnifiedHighlighter.java

@@ -80,7 +80,7 @@ public OGKUnifiedHighlighter(RuntimeEnvironment env,
         super(indexSearcher, indexAnalyzer);
 
         if (env == null) {
-            throw new IllegalArgumentException("`env' is null");
+            throw new IllegalArgumentException("env is null");
         }
         this.env = env;
     }

src/org/opensolaris/opengrok/search/context/PassageConverter.java

@@ -43,7 +43,7 @@ public class PassageConverter {
      */
     public PassageConverter(ContextArgs args) {
         if (args == null) {
-            throw new IllegalArgumentException("`args' is null");
+            throw new IllegalArgumentException("args is null");
         }
         this.args = args;
     }

src/org/opensolaris/opengrok/search/context/StrictLineBreakIterator.java

@@ -64,7 +64,7 @@ public int last() {
     @Override
     public int next(int n) {
         if (n < 0) {
-            throw new IllegalArgumentException("`n' cannot be negative");
+            throw new IllegalArgumentException("n cannot be negative");
         }
 
         int noff = current();
@@ -195,7 +195,7 @@ public CharacterIterator getText() {
     @Override
     public void setText(CharacterIterator newText) {
         if (newText == null) {
-            throw new IllegalArgumentException("`newText' is null");
+            throw new IllegalArgumentException("newText is null");
         }
         this.charIt = newText;
         this.breaks.clear();