Don't repeatably call str.upper() when testing method name.

JNRowe · JNRowe · commit 4b5edf4e349d · 2011-12-20T02:40:17.000Z
diff --git a/github2/core.py b/github2/core.py
@@ -141,14 +141,14 @@ def make_request(self, command, *args, **kwargs):
         page = kwargs.pop("page", 1)
         if page and not page == 1:
             post_data["page"] = page
-        method = kwargs.get("method", "GET")
-        if method.upper() == "POST" or method.upper() == "GET" and post_data:
+        method = kwargs.get("method", "GET").upper()
+        if method == "POST" or method == "GET" and post_data:
             response = self.request.post(self.domain, command, *args,
                                          **post_data)
-        elif method.upper() == "PUT":
+        elif method == "PUT":
             response = self.request.put(self.domain, command, *args,
                                         **post_data)
-        elif method.upper() == "DELETE":
+        elif method == "DELETE":
             response = self.request.delete(self.domain, command, *args,
                                            **post_data)
         else:
