Tickets :: Comment on [#12097] Export of mailfolders fails with cyrus imap

* Your Email Address
* Spam protection	Enter the letters below: .__.. .. ,. .. . \| \|\|\ \| \./ \| \|\| \| \|__\\| \\| \| \|__\|\|__\|
Comment	> See also (possibly) Ticket #12031 > >> You may say: This is also not solving the origin problem but I think >> as long as you cannot be sure the method returns integer values in >> all cases you have to check the result in calling processes. > > I disagree. > > For a general purpose API call that is exposed to any users (in other > words, where you don't control the calling code) it may make sense to > be super liberal in what you accept. > > But for a specific API/function call that really is only meant to be > called within the library, this is a bad idea because: > > 1) You control the input and should guarantee the data is correct here > 2) Because if you cant guarantee #1, that means there is a bug in our > code that needs to be fixed (and may be causing issues elsewhere) > 3) You are just adding complexity and performance hits when not necessary
Attachment
Watch this ticket