[0/10] Launcher app bundles on OS X
C.W. Betts
computers57 at hotmail.com
Mon Mar 18 15:13:26 CDT 2013
A quick glance at the source revealed that you removed static from some functions. Why? If a function isn't being called outside a source file, it's better to have it be static. This includes functions referenced in structs (but not the structs themselves).
A more seasoned developer is welcome to correct me if I'm wrong.
On Mar 17, 2013, at 11:18 AM, Per Johansson <per at morth.org> wrote:
> I'm resending this now after testing it with the Mac driver.
> They work well together, the launcher bundle will first briefly
> appear in the dock, then exit, and the mac driver icon appears.
>
> It's split it up into parts for easier review. I've also tried
> to make sure that even if e.g. part 6 fails review, part 1 - 5
> can still be applied.
>
> File type associations are not included because I plan to move
> that code into the mac driver but haven't gotten around to it yet.
>
> Regards,
> --
> Per Johansson
>
>
>
More information about the wine-devel
mailing list