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:37:33 UTC

[incubator-weex] branch release/0.24 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 release/0.24
in repository https://gitbox.apache.org/repos/asf/incubator-weex.git


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

commit b5c404a33db9b1daf9a2aa63ed273cb2cd604797
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)
   }
 
   /**