re: Added initial version of Wine OleView
Hi Piotr! Say, does it *do* anything useful yet? Might be worth waiting until it does before submitting it. - Dan -- Wine for Windows ISVs: http://kegel.com/wine/isv
On 6/4/06, Dan Kegel <dank(a)kegel.com> wrote:
Hi Piotr!
Say, does it *do* anything useful yet? Might be worth waiting until it does before submitting it. - Dan
I advised Piotr to send in small-sized patches that implement a feature at a time. For example, the first patch would just add enough functionality to create a window, and possibly the menu bar. The next couple of patches would start implementing the functionality. Of course, all these patches can be sent at once, just one feature per patch to keep patch size down. -- James Hawkins
On 6/4/06, James Hawkins <truiken(a)gmail.com> wrote:
Say, does it *do* anything useful yet?
I advised Piotr to send in small-sized patches that implement a feature at a time. For example, the first patch would just add enough functionality to create a window, and possibly the menu bar. The next couple of patches would start implementing the functionality.
I see. It would help if he said in the preamble to the patch what functionality the patch adds. In this case it would have been "just adds skeleton, can't even create a window". IMHO that's not really enough functionality to be worth submitting by itsellf, but if part of a series, e.g. "OleView (patch 1 of 5)", it would be fine. - Dan -- Wine for Windows ISVs: http://kegel.com/wine/isv
participants (2)
-
Dan Kegel -
James Hawkins