From: Hiroo H. <hir...@co...> - 2004-08-16 02:12:57
|
Rib> As I was making that last commit I started thinking about patch Rib> editors, and I had the idea to move the hasEditor and editPatch Rib> functions from IPatchDriver to IPatch. We could easily put a method in Rib> Patch to maintain compatibility with existing drivers. The advantage Rib> I see is that when you create an xml based editor, you can simply say Rib> what patch type it supports. Then when you click to edit a patch, it Rib> could give you a list of possible editors, or if there's only one open Rib> it right away. How does this sound? First I thought you were proposing 'IPatch.editPatch()', but it seems you were not. Could you give me an example which would help me to understand. BTW there are many lines like 'patch.getDriver().xxxxx(patch)'. They can be 'patch.xxxxx()' by defining some one line methods in IPatch. May I make the change? -- Hiroo Hayashi |