This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: setup
- From: Achim Gratz <Stromeko at nexgo dot de>
- To: cygwin-apps at cygwin dot com
- Date: Thu, 02 Jul 2015 23:03:11 +0200
- Subject: Re: setup
- Authentication-results: sourceware.org; auth=none
- References: <20150610185417 dot GL31537 at calimero dot vinschen dot de> <87wpzbs2yj dot fsf at Rainer dot invalid> <87mw060xg3 dot fsf at Rainer dot invalid> <20150612102945 dot GS31537 at calimero dot vinschen dot de> <87bngkzwki dot fsf at Rainer dot invalid> <87ioa8t0ha dot fsf at Rainer dot invalid> <20150629134154 dot GA2918 at calimero dot vinschen dot de> <873819okox dot fsf at Rainer dot invalid> <20150630164052 dot GE2918 at calimero dot vinschen dot de> <87vbe3ehrb dot fsf at Rainer dot invalid> <20150702091018 dot GA21212 at calimero dot vinschen dot de>
Corinna Vinschen writes:
> (*) This puzzles me a bit. You're keeping arrays and lists in terms of
> the file suffix (setup_ext, setup_ext_list), but you don't use the
> information here and elsewhere.
THat vector is a relatively late addition to the code when the rest was
already using literals as before. It's something that I want to clean
up, but not necessarily right now. As you noted, I'd probably need to
use a more structured data type.
> I'd prefer /* */ for multiline comments, but that's used pretty
> inconsistently anyway, so, never mind.
If you insistâ but setup is C++ and not C anyway.
> The existing code is inconsistently formatted, but for new code it would
> be nice if we could try to be more consistent. Always prepend a space
> to a left parenthesis, please.
I'll check that.
> Sorry if that's a lot. It just occured to me while reading your code.
> I'm not adamant about the structural change I outlined above, but to
> me it seems better to do it that way. What do you think?
It will just have to wait a bit, I think. But yes, these are all good
suggestions.
Regards,
Achim.
--
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+
SD adaptation for Waldorf microQ V2.22R2:
http://Synth.Stromeko.net/Downloads.html#WaldorfSDada