Some tools (mainly XML related) print filenames in format file:///filename.ext.
This patch adds support for this format in Open Selected command. Plus it should fix a buffer overflow.
The buffer overflow check is wrong since selectedFilename may have been moved from the beginning of targetFilename due to "file://" so there are less than MAX_PATH characters available.
Its easy to make this sort of mistake when working with fixed buffers, so to make this overflow safe it would be better to switch to a dynamic string type - SString or std::string or FilePath. Should also be separate patches, one for overflow safety and one for file:// as this makes it easier to work out why things have changed in source control.
You are right, appending new patch.