This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [setup PATCH] SetDlgItemFont


On Thu, 2003-07-31 at 22:20, Max Bowsher wrote:
> ChangeLog says it all really - this is just an incremental tweak to make it
> easier to create a "Finished" page, and to have all our font stylings in one
> place.
> 
> Note Gary had the SetDlgItemFont calls moved out in a seperate method.
> Given that they are almost the entire contents of the "case WM_INITDIALOG",
> I don't think this is appropriate.
> 
> Max.

Well, factoring out code for clarity is a good thing - the separate
function is appropriate.

Can you resubmit with that..
Cheers,
Rob


-- 
GPG key available at: http://members.aardvark.net.au/lifeless/keys.txt.
---

Attachment: signature.asc
Description: This is a digitally signed message part


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]