Change methods in RealImapFolder that perform a remote operation to never call open() themselves. It's up to the caller to make sure those methods are called on an open folder.
This doesn't significantly change the behavior of the app. In some cases we now avoid calling open() on an already open RealImapFolder, which triggered sending a NOOP command. So it removes a tiny bit of unnecessary network I/O, but not a lot.
I'll change it to requireOpen in a follow-up PR were the method also checks the required open mode. Some methods only require read access, others read-write access.
Change methods in
RealImapFolder
that perform a remote operation to never callopen()
themselves. It's up to the caller to make sure those methods are called on an open folder.This doesn't significantly change the behavior of the app. In some cases we now avoid calling
open()
on an already openRealImapFolder
, which triggered sending aNOOP
command. So it removes a tiny bit of unnecessary network I/O, but not a lot.