If someone can look over this patch, that would be helpful because it also looks like CLOSE also needs moved to the same block, but I don't want to start sending lots of patches and creating lots of noise on a list that's normally low volume.
Thanks Jim On 13/06/2016 23:46, NSS Ltd wrote: > I think there's life in AOX still and it's still worth working on. I > know working on a project when you don't get a lot of feedback and there > are other things in life can make it hard to know if the effort is > worthwhile - I think the positive responses are evidence that Arnt and > other's work is appreciated. > > I took a look through the debug logs from my recently updated server and > I see there are also errors being generated for a legitimate UNSELECT > command. AOX is complaining a mailbox is not specified when this > command does not use a mailbox. > > I've made a patch (attached) which fixes this by moving the UNSELECT > parsing in command.cpp [to where I believe it should really be] and does > an optional tidy up in the unselect.cpp (this second part is more a > personal preference to make it consistent with other commands code style). > > Jim >
