Ok.. looking at the source, wouldn't it make more sense to change the boolean variable, just changing the check won't change functionality, it probably would make more sense to change this line:
bEnableAutoCopySelection = (::GetPrivateProfileInt(PLUGIN_NAME, AUTO_COPY_SELECTION, (UINT)bEnableAutoCopySelection, iniFilePath) == 1);
bEnableAutoCopySelection= (::GetPrivateProfileInt(PLUGIN_NAME, AUTO_COPY_SELECTION, 0, iniFilePath) == 1);
If that does not work, then I would assume the value is being read in from some ini file because internally it is false and the check should follow the boolean.
of course we could just force it's hand to:
Only thing is I hate plugging a problem like this where something odd is going on that could crop up in other places.. what do you think?