Mass conversion to better log.debug use
authorJack Miller <jack@codezen.org>
Fri, 5 Jun 2015 00:51:32 +0000 (19:51 -0500)
committerJack Miller <jack@codezen.org>
Fri, 5 Jun 2015 01:10:06 +0000 (20:10 -0500)
Pass the format arguments to log.debug so we avoid the CPU hit
formatting strings we won't use.

14 files changed:
canto_curses/command.py
canto_curses/config.py
canto_curses/gui.py
canto_curses/guibase.py
canto_curses/html.py
canto_curses/input.py
canto_curses/main.py
canto_curses/screen.py
canto_curses/story.py
canto_curses/tag.py
canto_curses/tagcore.py
canto_curses/taglist.py
canto_curses/theme.py
plugins/smartlink.py

index fe45cfa..925276f 100644 (file)
@@ -190,7 +190,7 @@ def _unalias(lookup):
     # deref -1 for latest register, 1 for longform instead of obj
     total = total.replace(longest_alias, aliases[longest_alias][-1].longform, 1)
 
-    log.debug("Unaliased to: %s" % total)
+    log.debug("Unaliased to: %s", total)
 
     return shlex.split(total)
 
@@ -235,7 +235,7 @@ def cmd_complete_info():
         c = list(cmds.keys())
         c.extend(list(aliases.keys()))
         c.sort()
-        log.debug("CMDS: %s" % c)
+        log.debug("CMDS: %s", c)
         return ("", "", c)
     else:
         lookup, sig = _get_max_sig(lookup)
@@ -412,7 +412,7 @@ def _int_range(name, itrs, syms, fallback, s):
 
     if not rlist:
         rlist = fallback
-        log.debug("%s falling back to %s" % (rlist, fallback))
+        log.debug("%s falling back to %s", rlist, fallback)
 
     # If our fallback was empty, fail it.
     if not rlist:
@@ -481,7 +481,7 @@ class CommandHandler(PluginHandler):
             keyname += chr(k)
             optname += self.translate_key(keyname)
 
-        log.debug("trying key: %s" % optname)
+        log.debug("trying key: %s", optname)
 
         try:
             r = self.callbacks["get_opt"](optname)
index c489248..14f1972 100644 (file)
@@ -486,7 +486,7 @@ class CantoCursesConfig(SubThread):
         # Ensure all settings exist
         for setting in [ "border", "maxwidth", "maxheight", "align", "float" ]:
             if setting not in val:
-                log.debug("Couldn't find %s setting" % setting)
+                log.debug("Couldn't find %s setting", setting)
                 val[setting] = d[setting]
 
         # Ensure all settings are in their correct range
@@ -758,7 +758,7 @@ class CantoCursesConfig(SubThread):
 
     @write_lock(config_lock)
     def prot_configs(self, given, write = False):
-        log.debug("prot_configs given:\n%s\n" % json.dumps(given, indent=4, sort_keys=True))
+        log.debug("prot_configs given:\n%s\n", json.dumps(given, indent=4, sort_keys=True))
 
         if "tags" in given:
             for tag in list(given["tags"].keys()):
@@ -855,7 +855,7 @@ class CantoCursesConfig(SubThread):
                 # tag already, substitute the default template.
 
                 if tag not in self.tag_config:
-                    log.debug("Using default tag config for %s" % tag)
+                    log.debug("Using default tag config for %s", tag)
                     self.tag_config[tag] = self.tag_template_config.copy()
 
                 self.vars["strtags"].append(tag)
@@ -869,7 +869,7 @@ class CantoCursesConfig(SubThread):
         self.set_conf(c)
 
         for tag in newtags:
-            log.debug("New tag %s" % tag)
+            log.debug("New tag %s", tag)
             call_hook("curses_new_tag", [ tag ])
 
         self.eval_tags()
@@ -927,7 +927,7 @@ class CantoCursesConfig(SubThread):
         # If evaluated tags differ, we need to let other know.
 
         if prevtags != self.vars["curtags"]:
-            log.debug("Evaluated Tags Changed:\n%s\n" % json.dumps(self.vars["curtags"], indent=4))
+            log.debug("Evaluated Tags Changed:\n%s\n", json.dumps(self.vars["curtags"], indent=4))
             call_hook("curses_eval_tags_changed", [])
 
     def set_var(self, tweak, value):
@@ -966,7 +966,7 @@ class CantoCursesConfig(SubThread):
 
         for f in d_f:
             if f["name"] == name:
-                log.debug("updating %s with %s" % (f, conf))
+                log.debug("updating %s with %s", f, conf)
                 f.update(conf)
                 break
         else:
index 1124102..69c97d2 100644 (file)
@@ -200,7 +200,7 @@ class CantoCursesGui(CommandHandler):
                 continue
 
             cmds = self.cmdsplit(cmd)
-            log.debug("Resolved to %s" % cmds)
+            log.debug("Resolved to %s", cmds)
 
             # Now actually issue the commands
 
@@ -211,7 +211,7 @@ class CantoCursesGui(CommandHandler):
                 # Command is our one hardcoded command because it's special, and also shouldn't invoke itself.
                 if cmd == "command":
                     subcmd = self.screen.input_callback(':')
-                    log.debug("Got %s from user command" % subcmd)
+                    log.debug("Got %s from user command", subcmd)
                     subcmds = self.cmdsplit(subcmd)
                     for subcmd in subcmds:
                         okay = self.issue_cmd(subcmd)
index 6bd97a8..732534d 100644 (file)
@@ -162,7 +162,7 @@ class GuiBase(CommandHandler):
         loc_args = self.callbacks["get_var"]("location")
         argv = [argv[0]] + loc_args + argv[1:]
 
-        log.debug("Calling remote: %s" % argv)
+        log.debug("Calling remote: %s", argv)
 
         # check_output return bytes, we must decode.
         out = subprocess.check_output(argv).decode()
@@ -236,10 +236,10 @@ class GuiBase(CommandHandler):
                 return False
         else:
             if key in c[opt]["key"] and c[opt]["key"][key] and not overwrite:
-                log.debug("%s already bound to %s" % (key, c[opt]["key"][key]))
+                log.debug("%s already bound to %s", key, c[opt]["key"][key])
                 return False
 
-            log.debug("Binding %s.%s to %s" % (opt, key, cmd))
+            log.debug("Binding %s.%s to %s", opt, key, cmd)
 
             c[opt]["key"][key] = cmd
             self.callbacks["set_conf"](c)
@@ -327,7 +327,7 @@ class GuiBase(CommandHandler):
         return (possibles, lambda x : (True, x))
 
     def cmd_set(self, opt, val):
-        log.debug("SET: %s '%s'" % (opt, val))
+        log.debug("SET: %s '%s'", opt, val)
 
         evaluate = needs_eval(opt)
 
index 789449a..047140c 100644 (file)
@@ -49,7 +49,7 @@ class CantoHTML(HTMLParser):
             text = text.replace("\n", " ")
 
         if self.link_open:
-            log.debug("adding %s to link_text" % text)
+            log.debug("adding %s to link_text", text)
             self.link_text += text
 
         self.result += text
index e2bb3ef..ad78f45 100644 (file)
@@ -88,7 +88,7 @@ class InputBox(GuiBase):
         if complist:
             s += complist[0]
 
-        log.debug("printing: '%s'" % s[-1 * (maxx - self.minx):])
+        log.debug("printing: '%s'", s[-1 * (maxx - self.minx):])
         try:
             self.pad.addstr(s[-1 * (maxx - self.minx):])
         except:
index defc53d..304c526 100644 (file)
@@ -183,7 +183,7 @@ class CantoCurses(CantoClient):
                 pid, status = os.waitpid(-1, os.WNOHANG)
                 if pid == 0:
                     break
-                log.debug("CHLD %d has died: %d" % (pid, status))
+                log.debug("CHLD %d has died: %d", pid, status)
         except Exception as e:
             if e.errno == errno.ECHILD:
                 log.debug("CHLD no children?")
index c4e3b04..45b6512 100644 (file)
@@ -150,10 +150,10 @@ class Screen(CommandHandler):
 
             try:
                 curses.init_pair(i, fg, bg)
-                log.debug("color pair %s : %s %s" % (i, fg, bg))
+                log.debug("color pair %s : %s %s", i, fg, bg)
             except:
-                log.debug("color pair failed!: %d fg: %d bg: %d" %
-                        (i + 1, fg, bg))
+                log.debug("color pair failed!: %d fg: %d bg: %d"
+                        i + 1, fg, bg)
         return 0
 
     def screen_opt_change(self, conf):
@@ -268,8 +268,8 @@ class Screen(CommandHandler):
         # Height + 1 to account for the last curses pad line
         # not being fully writable.
 
-        log.debug("h: %s w: %s" % (self.height, self.width))
-        log.debug("h: %s w: %s" % (height, width))
+        log.debug("h: %s w: %s", self.height, self.width)
+        log.debug("h: %s w: %s", height, width)
         pad = curses.newpad(height + 1, width)
 
         # Pass on callbacks we were given from CantoCursesGui
@@ -432,7 +432,7 @@ class Screen(CommandHandler):
 
         # Cleanup any window objects that will be destroyed.
         for w in self.windows:
-            log.debug("die to %s" % w)
+            log.debug("die to %s", w)
             w.die()
 
         self.floats = []
@@ -601,10 +601,10 @@ class Screen(CommandHandler):
 
         comp = self.input_box.break_completion()
         if comp:
-            log.debug("inserting: %s" % comp)
+            log.debug("inserting: %s", comp)
             readline.insert_text(comp)
 
-        log.debug("KEY: %s" % r)
+        log.debug("KEY: %s", r)
         return r
 
     def _exception_wrap(self, fn, *args):
@@ -667,7 +667,7 @@ class Screen(CommandHandler):
 
     def _focus(self, win):
         self.focused = win
-        log.debug("Focusing window (%s)" % (self.focused,))
+        log.debug("Focusing window (%s)", self.focused)
 
     def type_color_idx(self):
         def ci(x):
@@ -706,7 +706,7 @@ class Screen(CommandHandler):
     def cmd_color(self, idx, fg, bg):
         conf = self.callbacks["get_conf"]()
 
-        log.debug(":COLOR IDX: %s" % idx)
+        log.debug(":COLOR IDX: %s", idx)
         if idx in ['deffg', 'defbg']:
             conf["color"][idx] = fg  # Ignore second color pair
         else:
@@ -717,7 +717,7 @@ class Screen(CommandHandler):
                 color['bg'] = bg
             conf["color"][idx] = color
 
-        log.debug("color %s set: %s" % (idx, conf["color"][idx]))
+        log.debug("color %s set: %s", idx, conf["color"][idx])
 
         self.callbacks["set_conf"](conf)
 
index 3a1202a..2d510ff 100644 (file)
@@ -133,7 +133,7 @@ class Story(PluginHandler):
                 if attr not in self.content:
                     needed_attrs.append(attr)
             if needed_attrs:
-                log.debug("%s needs: %s" % (self.id, needed_attrs))
+                log.debug("%s needs: %s", self.id, needed_attrs)
                 tag_updater.need_attributes(self.id, needed_attrs)
 
         # All other story options are formats / enumerations, redraw.
@@ -301,7 +301,7 @@ class Story(PluginHandler):
                 self.need_refresh()
                 self.callbacks["release_gui"]()
 
-                log.debug("%s still needs %s" % (self, attr))
+                log.debug("%s still needs %s", self, attr)
 
                 self.left = " "
                 self.left_more = " "
index 6211481..42b68c4 100644 (file)
@@ -112,7 +112,7 @@ class Tag(PluginHandler, list):
         self.update_plugin_lookups()
 
     def die(self):
-        log.debug("tag %s die()" % self.tag)
+        log.debug("tag %s die()", self.tag)
         # Reset so items get die() called and everything
         # else is notified about items disappearing.
 
index e325d33..143a8c5 100644 (file)
@@ -62,7 +62,7 @@ class TagCore(list):
 
         for idx, id in enumerate(self[:]):
             if id in ids:
-                log.debug("removing: %s" % (id,))
+                log.debug("removing: %s", id)
 
                 list.remove(self, id)
                 removed.append(id)
@@ -170,7 +170,7 @@ class TagUpdater(SubThread):
         self.lock.acquire_write()
         for item in items:
             if tagcore and item.id in tc:
-                log.debug("%s still in tagcore, not removing" % item.id)
+                log.debug("%s still in tagcore, not removing", item.id)
                 continue
             if item.id in self.attributes:
                 del self.attributes[item.id]
index f330da1..e77ecde 100644 (file)
@@ -66,7 +66,7 @@ class TagList(GuiBase):
                 if tagobj.tag == tagcore.tag:
                     break
             else:
-                log.debug("Instantiating Tag() for %s" % tagcore.tag)
+                log.debug("Instantiating Tag() for %s", tagcore.tag)
                 Tag(tagcore, self.callbacks)
 
         self.update_tag_lists()
@@ -289,7 +289,7 @@ class TagList(GuiBase):
         return (["read","marked","-read","-marked"], lambda x : (True, x))
 
     def on_new_tagcore(self, tagcore):
-        log.debug("Instantiating Tag() for %s" % tagcore.tag)
+        log.debug("Instantiating Tag() for %s", tagcore.tag)
         Tag(tagcore, self.callbacks)
 
     def on_del_tagcore(self, tagcore):
@@ -340,7 +340,7 @@ class TagList(GuiBase):
                 tag_updater.need_attributes(item, sa)
 
     def cmd_goto(self, items):
-        log.debug("GOTO: %s" % items)
+        log.debug("GOTO: %s", items)
         self._goto([item.content["link"] for item in items])
 
     def cmd_tag_state(self, state, tags):
@@ -647,7 +647,7 @@ class TagList(GuiBase):
     def cmd_promote(self, tags):
         for tag in tags:
 
-            log.debug("Promoting %s\n" % tag.tag)
+            log.debug("Promoting %s\n", tag.tag)
 
             # Refetch because a promote call will cause our eval_tag hook to
             # recreate visible_tags.
@@ -1083,7 +1083,7 @@ class TagList(GuiBase):
         return (main_offset, curpos + lines)
 
     def redraw(self):
-        log.debug("Taglist REDRAW (%s)!\n" % self.width)
+        log.debug("Taglist REDRAW (%s)!\n", self.width)
         self.pad.erase()
 
         target_obj = self.callbacks["get_var"]("target_obj")
index 00f8b3a..8955bb9 100644 (file)
@@ -124,7 +124,7 @@ def theme_print_one(pad, uni, width):
             try:
                 pad.waddch(ec)
             except:
-                log.debug("Can't print escaped ec: %s in: %s" % (ec, uni))
+                log.debug("Can't print escaped ec: %s in: %s", ec, uni)
 
             width -= cwidth
             escaped = False
@@ -225,8 +225,8 @@ def theme_print_one(pad, uni, width):
             try:
                 pad.waddch(ec)
             except Exception as e:
-                log.debug("Can't print ec: %s in: %s" % (ec, repr(encoder(uni))))
-                log.debug("Exception: %s" % e)
+                log.debug("Can't print ec: %s in: %s", ec, repr(encoder(uni)))
+                log.debug("Exception: %s", e)
 
             width -= cwidth
 
@@ -252,7 +252,7 @@ def theme_print(pad, uni, mwidth, pre = "", post = "", cursorbash=True, clear=Tr
         try:
             pad.move(y, (mwidth - postl))
         except:
-            log.debug("move error: %d %d" % (y, mwidth - postl))
+            log.debug("move error: %d %d", y, mwidth - postl)
         theme_print_one(pad, post, postl)
 
     if cursorbash:
index 7a9b232..df1a962 100644 (file)
@@ -130,7 +130,7 @@ class SmartLinkThread(Thread):
                         file_handlers = []
 
             for f_handler in file_handlers:
-                log.debug("f_handler: %s" % f_handler)
+                log.debug("f_handler: %s", f_handler)
                 got_handler = self.try_handler(f_handler, "file", fileoutput)
                 if got_handler:
                     self.href = tmpnam
@@ -220,7 +220,7 @@ class SmartLinkThread(Thread):
         tmpdir = tempfile.mkdtemp(prefix="canto-")
         tmpnam = tmpdir + '/' + fname
 
-        log.debug("Downloading %s to %s" % (self.href, tmpnam))
+        log.debug("Downloading %s to %s", self.href, tmpnam)
 
         on_hook("curses_exit", lambda : (os.unlink(tmpnam)))
         on_hook("curses_exit", lambda : (os.rmdir(tmpdir)))