Well, for the future I have planned grouping and automatic addition of #ifdef HAVE_ and that sort of things. That is overdoing a little perhaps but it will look nicer. :-)
This is that it already does I think.
Please try it out yourself.
and the operation is really one time.
It depends what you mean by one time. People make mistakes. winapi_cleanup does C++ comment to C comment conversion and this is "one time" as well...
I think it is better than Alexandre runs winapi_cleanup --include-quotes himself if he thinks it does the correct things. If not please inform me what the problem is.
In my tree, the patch is over 800k.
On December 31, 2002 06:41 am, Patrik Stridvall wrote:
Well, that's cool, but your original message gave the impression that this is still a work in progress not ready to be used. Which it seems is the wrong impression. Just let Alexandre know to run the above mentioned command after applying your patch, that will do.
BTW, did you compile Wine after this patch? If it compiles, then I don't think there is any problem, no? In fact, if it compiles we are *guaranteed* that the result is correct because we don't have any duplicated header filename:
[dimi@dimi wine.src]$ find -name "*.h" | sed 's%.*/%%' | sort -u | uniq -c | sort -rn | head -1 1 zmouse.h
On Tue, Dec 31, 2002 at 01:30:10PM -0500, Dimitrie O. Paun wrote:
[dimi@dimi wine.src]$ find -name "*.h" | sed 's%.*/%%' | sort -u | uniq -c | sort -rn | head -1
Isn't the result of sort -u | uniq -c always either 1 or nothing at all?
ciao Jörg
-- Joerg Mayer jmayer@loplof.de I found out that "pro" means "instead of" (as in proconsul). Now I know what proactive means.
On January 1, 2003 12:39 am, Joerg Mayer wrote:
Isn't the result of sort -u | uniq -c always either 1 or nothing at all?
But of course, silly me:
[dimi@dimi wine.src]$ find -name "*.h" | sed 's%.*/%%' | sort | uniq -c | sort -rn | head -15 11 main.h 4 user.h 4 resource.h 3 utils.h 3 license.h 3 hal.h 2 unicode.h 2 thread.h 2 shlobj.h 2 regproc.h 2 process.h 2 parser.h 2 dialog.h 2 async.h 1 zmouse.h
Anyway, only a few headers are duplicated, so they should be simple to check.
"Patrik Stridvall" ps@leissner.se writes:
I'm not sure there's much point in fixing all the C files. It works fine the way it is now, the only thing that we need the change is the exported headers.