By now, a Segwit version would be better.
BIP38 with segwit addresses would be non-standard, because it only mentions encoding the address into compressed or uncompressed base58.
I don't think anyone actually uses BIP38 anymore outside the collectibles community. There are better ways to encrypt large amounts of private keys especially the ones that are derived from HD keys.
PS: Your algorithm for deriving a brainwallet is quite hard to remember
