[Webkit-unassigned] [Bug 40909] Support WTF::Vector pretty printer for GDB 7

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 22 22:18:17 PDT 2010


Shinichiro Hamaji <hamaji at chromium.org> changed:

           What    |Removed                     |Added
  Attachment #59338|review?                     |review-
               Flag|                            |

--- Comment #5 from Shinichiro Hamaji <hamaji at chromium.org>  2010-07-22 22:18:17 PST ---
(From update of attachment 59338)
r- for minor issues.

 +  build_pritty_printers_dict()

 +  pretty_printers_dict = {}
It would be better not to use a global variable? Maybe

pretty_printers = build_pretty_printers()
gdb.pretty_printers.append(lambda val: lookup_function(pretty_printers, val))

(I'm not 100% sure if it's OK to pass lambda to gdb.pretty_printers though)

 +      "Print a WTF::Vector"
Could you add a document about how each members will be used? I guess both children and to_string will be called for GDB's p command when display_hint is 'array'? Maybe putting URL for GDB's python document would be nice.

 +              elt = self.item.dereference()
I think element would be better. Usually WebKit code doesn't like abbreviations.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list