From 2d8ec60eedecc0aa53d14a41f763719bc6288002 Mon Sep 17 00:00:00 2001
From: Kevin Sawicki <kevinsawicki@gmail.com>
Date: Mon, 20 Jun 2016 17:14:16 -0700
Subject: [PATCH] Use const/let instead of var

---
 lib/browser/api/menu-item.js | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/lib/browser/api/menu-item.js b/lib/browser/api/menu-item.js
index 5ff9676..15ede39 100644
--- a/lib/browser/api/menu-item.js
+++ b/lib/browser/api/menu-item.js
@@ -1,11 +1,9 @@
 'use strict'
 
-var MenuItem, methodInBrowserWindow, nextCommandId, rolesMap
-
-nextCommandId = 0
+let nextCommandId = 0
 
 // Maps role to methods of webContents
-rolesMap = {
+const rolesMap = {
   undo: 'undo',
   redo: 'redo',
   cut: 'cut',
@@ -20,7 +18,7 @@ rolesMap = {
 }
 
 // Maps methods that should be called directly on the BrowserWindow instance
-methodInBrowserWindow = {
+const methodInBrowserWindow = {
   minimize: true,
   close: true
 }
@@ -29,11 +27,11 @@ const methodInApp = {
   quit: true
 }
 
-MenuItem = (function () {
+const MenuItem = (function () {
   MenuItem.types = ['normal', 'separator', 'submenu', 'checkbox', 'radio']
 
   function MenuItem (options) {
-    var click, ref
+    let click, ref
     const {app, Menu} = require('electron')
     click = options.click
     this.selector = options.selector
@@ -72,7 +70,7 @@ MenuItem = (function () {
     this.commandId = ++nextCommandId
     this.click = (focusedWindow) => {
       // Manually flip the checked flags when clicked.
-      var methodName, ref1, ref2
+      let methodName, ref1, ref2
       if ((ref1 = this.type) === 'checkbox' || ref1 === 'radio') {
         this.checked = !this.checked
       }
-- 
2.7.4