Merge pull request #10093 from MarcelGerber/max-code-hints-fixup

Fixup of maxCodeHints pref
adobe · Dec 10, 2014 · 5e6b9cf · 5e6b9cf
2 parents d9bc65e + 2af9c56
commit 5e6b9cf
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/editor/CodeHintList.js b/src/editor/CodeHintList.js
@@ -44,7 +44,7 @@ define(function (require, exports, module) {
      * @constructor
      * @param {Editor} editor
      * @param {boolean} insertHintOnTab Whether pressing tab inserts the selected hint
-     * @param {number} maxResults Maximum hints displayed at once. Defaults to 1000
+     * @param {number} maxResults Maximum hints displayed at once. Defaults to 50
      */
     function CodeHintList(editor, insertHintOnTab, maxResults) {
 
@@ -67,10 +67,7 @@ define(function (require, exports, module) {
          *
          * @type {number}
          */
-        this.maxResults = Math.min(
-            (maxResults > 0 && ValidationUtils.isInteger(maxResults) && maxResults) || 1000,
-            1000
-        );
+        this.maxResults = ValidationUtils.isIntegerInRange(maxResults, 1, 1000) ? maxResults : 50;
 
         /**
          * Is the list currently open?