Thank you Andrew for the review, I definitely missed a few important things. On 12/11/2022 02:41, Allan McRae wrote:
Thanks for the reviews. I had noticed the -U <url> issue the other night and am trying to figure that out. It falls into the "not easy" category. I _think_ we might be able to work-around these cases by defining a new type of event passed from the child to the parent in that case.
I can make a merge request, but don't expect to have a chance to deal with some of the comments you made in the next couple of weeks. I'm happy for anyone to provide fix-up patches.
With some guidance I should be able to provide patches for most of the comments, if not all. It might take a week or so, so I'll post a message when I can start on it so we don't duplicate the work in case you beat me to it. Remi