Works fine here (w/HEAD & 4.1).  getMatchingIdentity() and 
hasAddress() *already* require bare e-mail addresses according to 
their documentation and, sure enough, both message.php and mailbox.php 
do all the pre-processing needed to remove all of the personal 
information from an e-mail string.

If a user decides to change it's 'from: address' to

username <user@example.com>

instead of simply

user@example.com

no messages are marked any more as 'personal' in mailbox.php and

message.php.



This problem could be solved by changing in imp/lib/Identity/imp.php the

getMatchingIdentity and hasAddress functions.



in getMatchingIdentity, instead of

             /* Next, search all from addresses. */

             if (isset($own_addresses[$find_address])) {

                 return $own_addresses[$find_address];

             }

we could use

             foreach (array_keys($own_addresses) as $current_address) {

                 if (IMP::bareAddress($current_address) == $find_address)

{

                     return true;

                 }

             }



and in hasAddress($address), instead of

//        return isset($list[$address]);

we could use

         foreach (array_keys($list) as $current_address) {

            if (IMP::bareAddress($current_address) == $address) {

                 return true;

            }

         }

         return false;



I observed this in IMP 4.0.3, but after a look in the cvs, I have the 
impression that it should also apply to all later versions (including 
cvs).