Andre Poenitz wrote:
On Thu, Feb 19, 2009 at 12:30:55PM -0500, rgheck wrote:
This patch is pretty simple: It just moves some routines needed to sync up the modules into the newish class LayoutModuleList. I think this is pretty straightforward, and it seems to work, but I thought I'd post it before committing in case anyone has any other thoughts.


Index: src/LayoutModuleList.h
===================================================================
--- src/LayoutModuleList.h      (revision 28558)
+++ src/LayoutModuleList.h      (working copy)
@@ -13,10 +13,13 @@
 #define LAYOUTMODULELIST_H
#include <list>
+#include <set>
 #include <string>

I see now that you already managed to sneak in a #include <set>
into BufferParams.h in change 25917, so I can't really complain
on this one ;-}

There's no real reason I have to use set there. I think I could just as well use list, though I don't need the list features. Worth doing?

rh

Andre'

Reply via email to