Re: [1.4.20RC2] case exact match in decodeheader
Takahiro Kambe <[email protected]>
| Newsgroups | gmane.mail.squirrelmail.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi, In message <[email protected]> on Sat, 3 Oct 2009 22:10:44 -0700, Paul Lesniewski <[email protected]> wrote: >> - if (preg_match('/=\?([^?]+)\?(q|b)\?([^?]+)\?=/', $ret)) >> + if (preg_match('/=\?([^?]+)\?(Q|B)\?([^?]+)\?=/Ui', $ret)) The patch was provided by obache at NetBSD.org. > Kanbe-san, thank you. Looks like I forgot to add the i modifier in a > couple places where eregi() was being replaced. However, unless you > can prove otherwise, I believe the U modifier does nothing here, so it Yes, I think that U modifier isn't needed unless the string is really correctly encoded and I don't have such a wrong case. Probably, U modified was added as other lines which make matching to MIME B/Q encoding. > is being left out. This has been committed. Thank you. Thanks for your quick action! -- Takahiro Kambe <[email protected]> ------------------------------------------------------------------------------ Come build with us! The BlackBerry® Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9-12, 2009. Register now! http://p.sf.net/sfu/devconf ----- squirrelmail-devel mailing list Posting guidelines: http://squirrelmail.org/postingguidelines List address: [email protected] List archives: http://news.gmane.org/gmane.mail.squirrelmail.devel List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel