[PATCH] Replace builtin WineFile Execute Dialog with standard RunFileDialog
Steven Edwards
winehacker at gmail.com
Thu Dec 24 01:46:01 CST 2009
Hi Dmitry,
On Thu, Dec 24, 2009 at 2:21 AM, Dmitry Timoshkov
<dmitry at codeweavers.com> wrote:
> Avoiding a needless renaming of ID_EXECUTE to ID_RUN would make the patch
> much smaller. Also avoiding useless typedef, making WineFile_OnRun() static,
> using correct casts, avoiding hungarian notation and magic flags would make
> the patch slightly cleaner.
Thanks for the feedback. I'll clean it up per your suggestions though
I don't think that renaming ID_EXECUTE is not needless since the
dialog containing the word is gone. I think it makes it less trouble
if your doing a cursory inspection to make it more consistent.
Have a Merry Christmas!
--
Steven Edwards
"There is one thing stronger than all the armies in the world, and
that is an idea whose time has come." - Victor Hugo
More information about the wine-devel
mailing list