You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@weex.apache.org by ky...@apache.org on 2019/04/25 09:34:36 UTC

[incubator-weex] branch master updated: [jsfm] Fix the lint error in Document (#2368)

This is an automated email from the ASF dual-hosted git repository.

kyork pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-weex.git


The following commit(s) were added to refs/heads/master by this push:
     new 708c97e  [jsfm] Fix the lint error in Document (#2368)
708c97e is described below

commit 708c97e36c2c534e0b0250c6069cc09caafe1280
Author: Hanks <zh...@gmail.com>
AuthorDate: Thu Apr 25 04:34:30 2019 -0500

    [jsfm] Fix the lint error in Document (#2368)
    
    Remove the useless temporary variable `result` in the `fireEvent` method.
---
 runtime/vdom/Document.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/runtime/vdom/Document.js b/runtime/vdom/Document.js
index 71145bd..8b5cf6e 100644
--- a/runtime/vdom/Document.js
+++ b/runtime/vdom/Document.js
@@ -183,14 +183,12 @@ export default class Document {
     if (domChanges) {
       updateElement(el, domChanges)
     }
-    let result
     let isBubble
     const $root = this.getRef('_root')
     if ($root && $root.attr) {
       isBubble = $root.attr['bubble'] === 'true'
     }
-    result = el.fireEvent(type, event, isBubble, options)
-    return result
+    return el.fireEvent(type, event, isBubble, options)
   }
 
   /**