This is the mail archive of the cygwin-patches 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] |
Hi David, On Feb 28 11:48, David Allsopp wrote: > Corinna Vinschen wrote: > > On Feb 27 17:13, David Allsopp wrote: > > > Corinna Vinschen wrote: > > > > On Feb 25 16:27, David Allsopp wrote: > > > > > This patch (below - I hope I have managed to format this email > > > > > correctly) alters the behaviour of dll_list::topsort to preserve > > > > > the order of dlopen'd units. > > > > > [...] > > > > > This patch is licensed under 2-clause BSD as per > > > > > winsup/CONTRIBUTORS, Copyright (c) 2017, MetaStack Solutions Ltd. > > > > Do you really want to make it (c) MetaStack? > > Oh, I was assuming that there would just be an implied mapping! Simple is best (MetaStack is just me, anyway!), so the previous patch and this fixup may be merged as my personal copyright, yes. > > <snip> > > if (loaded_dlls > 0) > > dlopen_deps = (dll**) cmalloc (HEAP_2_DLL, maxdeps*sizeof (dll*)); > > while ((d = d->next)) > > { > > [...] > > } > > if (dlopen_deps) > > cfree (dlopen_deps); > > > > Do you want to tweak your patch accordingly? > > That's much neater - attached is a fixup (which obviously looks a lot clearer with git diff --ignore-all-space) Applied and pushed. I took the liberty to add two formatting tweaks, as well as most of the description of your OP for the log message, and squashed this into a single patch. I'll uploaded new developer snapshots to https://cygwin.com/snapshots/ containing your patch for testing today. This might take a while since I have trouble with my cable provider and my upload speed is currently almost 0 :-P Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat
Attachment:
signature.asc
Description: PGP signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |