On 12-May-2002 Grahame Bowland wrote: > Hi, > > I've been meaning to learn C++ for a while, so I took a look > at the TODO file and built blackbox from CVS :) >
welcome (-: > I'm starting with the simple looking items. To remove the > current workspace from the window menu, the workspace needs > to be identified by some other data than the index into the > menu. In C I'd just put a "void *user_data" onto the > constructor to hold a pointer to anything. > > In the interests of creating a patch that will actually > be accepted into this shiny C++ code, what's the appropriate > way of doing this for blackbox? In this particular case we > only need to store another int; it's probably useful to be > able to store anything though. > Brad has new menu code in his testing branch. I am purposefully ignoring the menu until we get it ported over. If you would like to help and are new to X coding and C++ perhaps you could help us as a fresh pair of eyes. Brad and I would like to be as const correct as possible. I have also been weeding out redundant variables and functions. Or perhaps you would like to send comment patches which document non obvious items. Basically any comments on the quality or sanity of the code are welcome, as long as they are constructive. On the subject of patches, I like diff -pruN style patches.
