From e94c8e810b9e8c59d33cf3dbeb7c88e8abbe33d9 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Wed, 9 Mar 2011 11:48:00 +0100 Subject: [PATCH] [IMP] use switch/case in QWeb.trim, remove default case (as it's already handled by conditional at function top) bzr revid: xmo@openerp.com-20110309104800-tdbzuqtg9lpardhr --- addons/base/static/qweb/qweb.js | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/addons/base/static/qweb/qweb.js b/addons/base/static/qweb/qweb.js index 22b1411b566..7f35ef9ceb0 100644 --- a/addons/base/static/qweb/qweb.js +++ b/addons/base/static/qweb/qweb.js @@ -81,14 +81,13 @@ var QWeb = { }, trim : function(v, mode) { if (!v || !mode) return v; - if (mode == 'both') { - return v.replace(/^\s*|\s*$/g, ""); - } else if (mode == "left") { - return v.replace(/^\s*/, ""); - } else if (mode == "right") { - return v.replace(/\s*$/, ""); - } else { - return v; + switch (mode) { + case 'both': + return v.replace(/^\s*|\s*$/g, ""); + case "left": + return v.replace(/^\s*/, ""); + case "right": + return v.replace(/\s*$/, ""); } }, escape_text:function(s) {