# HG changeset patch
# User Eris Caffee <discordia@eldalin.com>
# Date 1334728390 18000
# Node ID 713e3c5db1e1d54f783e30306368b55777c49532
# Parent  1b73e60d53b4632140605f467cc414060201bf49
just comments

diff -r 1b73e60d53b4 -r 713e3c5db1e1 include/App.h
--- a/include/App.h	Tue Apr 17 01:56:49 2012 -0500
+++ b/include/App.h	Wed Apr 18 00:53:10 2012 -0500
@@ -32,6 +32,16 @@
 
    /////////////////////////////////////
    // TODO: Need to be able to support child windows.
+   // Thus, I need a tree and this class should be a node of the tree.
+   // I could use an STL multimap container that maps child nodes onto parent nodes
+   // The root node would need to be a bit special.  Perhaps it would map to itself.
+   // Use the equal_range() method to get the children of the specified node.
+   // I would still need to implement my own tree walking methods: get root node,
+   // for each child (get children and recurse).
+   //
+   // It may be more efficient to implement the tree directly instead of using
+   // std::map as an intermediary.
+
    class WindowData
       {
       public:
diff -r 1b73e60d53b4 -r 713e3c5db1e1 src/App.cpp
--- a/src/App.cpp	Tue Apr 17 01:56:49 2012 -0500
+++ b/src/App.cpp	Wed Apr 18 00:53:10 2012 -0500
@@ -213,6 +213,13 @@
 	       break;
 	    case ButtonPress:
 	    case KeyPress:
+	       // Doing the lookup like this, using XLookupString, means I get the modified
+	       // character back.  Thus type shift+a gives me XK_A instead of XK_a
+	       // If I used XKeySym I'd instead get a sequence such as this:
+	       // XK_Shift_L followed by XK_a
+	       //
+	       // In the end, that might be a more useful way to go, but for now I'll keep
+	       // things simple.
 	       XLookupString(&(e.xkey), key_buffer, 7, &k, &compose_status);
 	       switch (k)
 		  {