Simplify code; no need to pass issue arg to GetApprovingReviewers().

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld."""
 
 import difflib
@@ skipping 665 matching lines @@
     self.has_patchset = False
 
   def GetMostRecentPatchset(self, issue):
     return self.RpcServer().get_issue_properties(
         int(issue), False)['patchsets'][-1]
 
   def GetPatchSetDiff(self, issue, patchset):
     return self.RpcServer().get(
         '/download/issue%s_%s.diff' % (issue, patchset))
 
-  def GetApprovingReviewers(self, issue):
+  def GetApprovingReviewers(self):
     return get_approving_reviewers(
-        self.RpcServer().get_issue_properties(int(issue), True))
+        self.RpcServer().get_issue_properties(self.GetIssue(), True))
 
   def SetIssue(self, issue):
     """Set this branch's issue.  If issue=0, clears the issue."""
     if issue:
       RunGit(['config', self._IssueSetting(), str(issue)])
       if self.rietveld_server:
         RunGit(['config', self._RietveldServer(), self.rietveld_server])
     else:
       RunGit(['config', '--unset', self._IssueSetting()])
       self.SetPatchset(0)
@@ skipping 871 matching lines @@
     else:
       print 'No description set.'
       print 'Visit %s/edit to set it.' % (cl.GetIssueURL())
       return 1
 
   # Keep a separate copy for the commit message, because the commit message
   # contains the link to the Rietveld issue, while the Rietveld message contains
   # the commit viewvc url.
   # Keep a separate copy for the commit message.
   if cl.GetIssue():
-    change_desc.update_reviewers(cl.GetApprovingReviewers(cl.GetIssue()))
+    change_desc.update_reviewers(cl.GetApprovingReviewers())
 
   commit_desc = ChangeDescription(change_desc.description)
   if cl.GetIssue():
     commit_desc.append_footer('Review URL: %s' % cl.GetIssueURL())
   if options.contributor:
     commit_desc.append_footer('Patch from %s.' % options.contributor)
 
   print 'Description:', repr(commit_desc.description)
 
   branches = [base_branch, cl.GetBranchRef()]
@@ skipping 586 matching lines @@
   GenUsage(parser, 'help')
   return CMDhelp(parser, argv)
 
 
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   colorama.init()
   sys.exit(main(sys.argv[1:]))