Manuel Camargo
Manuel Camargo
> The lower wrap limit was added because some people were having problems with PHP's mail() function breaking lines shorter than 75 chars, resulting in corrupt messages. We didn't find...
> Agreed, it should be consistent. Another problem within this encoding function is that it doesn't know its context, meaning that it doesn't/can't take the length of the label into...
Headings are also lost. Only lists and tables are preserved.
> Looks like a valid improvement. I do wonder if we could come up with a more readable group name. Maybe something like mbstringExtensionUnavailable, though that may be a bit...
@jrfnl I've been thinking on your proposal and to make a fully operative no-mbstring suite, we need to tag all the tests that are requiring mbstring to exclude them selectively...
Let me add 3 extra ones and a little thing I've noted before proceeding
I'm going to add one extra test `testHtmlIso8859` and also take advantage of this PR to add, apart from the group, the `@requires extension mbstring` as it is using both...
@jrfnl the idea is to group them to be able to exclude them in certain testing protocols to provide a cleaner testing report that focus on tests that are actually...
> I don't understand why you want to make changes to the test suite for **this** package just to accommodate some test set up in a completely unrelated package. @jrfnl...
@campbell-m @duckydan I assume that you are explicitly calling `parseAddresses` in your setup. For very basic parsing, `imap` is not required and everything will still be working. Note that this...