"Jacek Caban" jacek@codeweavers.com wrote:
return E_NOTIMPL;
- case QUERY_USES_HISTORYFOLDER:
default: return E_FAIL; }FIXME("Unsupported option QUERY_USES_HISTORYFOLDER\n");
shouldn't there be a "return E_NOTIMPL" after the FIXME as we do with the others?
It doesn't really matter IMO, so I've made the patch no-op. This way Wine will behave like IE6 until we will have proper IE7 implementation.
Then please add the explicit /* fall through */ comment indicating that the behaviour is intentional, and not just an oversight.