From a119f687dbea53b2b2b284b3582193db579003b0 Mon Sep 17 00:00:00 2001
From: Matthew Douglass <matthew@20xr.com>
Date: Wed, 29 Jul 2015 21:06:07 -0700
Subject: [PATCH] Keeps errors returned by libicu prepare
 (nodexmpp/node-stringprep#65)

Only attempt to use the javascript fallback routines when the native
libicu function is not available. This fixes two issues:
1) Overwriting the libicu error with a success from the fallback
2) Reporting a LIBICU_NOT_AVAILABLE error when fallbacks are disabled
and libicu is present, but returns an error
---
 index.js | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/index.js b/index.js
index c40657a..f6a048b 100644
--- a/index.js
+++ b/index.js
@@ -61,15 +61,14 @@ StringPrep.prototype.useJsFallbacks = true
 
 StringPrep.prototype.prepare = function(value) {
     this.value = value
-    try {
-        if (this.stringPrep) {
-            return this.stringPrep.prepare(this.value)
-        }
-    } catch (e) {}
-    if (false === this.useJsFallbacks) {
+
+    if (this.stringPrep) {
+        return this.stringPrep.prepare(this.value)
+    } else if (this.useJsFallbacks) {
+        return this.jsFallback()
+    } else {
         throw new Error(this.LIBICU_NOT_AVAILABLE)
     }
-    return this.jsFallback()
 }
 
 StringPrep.prototype.isNative = function() {