Hi Neil and Nicolas, I've made some changes and added some comments (this is also now in CR+LF SciTE standard and works with new snapshot)
http://mysite.mweb.co.za/residents/sdonovan/DirectorExtension.zip 'ipc.scite.name' will now always have the valid listening pipe name, whether set externally or generated internally - so 'PipeName' is redundant. 'current.pid' is actually useless as well as redundant; an internal extension can now look at 'ipc.scite.name' and find what they want from that. If we run out of available listening slots, then the request fails and the result is '*', which seems an unlikely valid filename ;) I appreciated Nicolas' idea that one could specify the file to receive the registered notify pipename, using the correspondent syntax. However, this logic fails if the file doesn't exist, so I'm using a separate open() call to emphasize that this is a regular temp file, and not a pipe. (If you _do_ use a pipe, it will get closed almost immediately). Any other issues? Shall we go with this? steve d. On 7/8/07, Neil Hodgson <[EMAIL PROTECTED]> wrote:
There seems to be some unnecessary entities. Looking at the set of properties mentioned in the DirectorExtension, ["ipc.director.name", "current.pid", "ipc.scite.name", "PipeName"] it appears that "PipeName" is just giving "ipc.scite.name" a default value so could go into ipc.scite.name (is there a property hierarchy issue here?). I don't see any use for "current.pid". AddSendPipe should error out a little more nicely by ignoring calls after s_send_cnt has reached MAX_PIPES. Neil
_______________________________________________ Scite-interest mailing list [email protected] http://mailman.lyra.org/mailman/listinfo/scite-interest
