This is the mail archive of the 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]

Re: cron allowed to interact with desktop ?


While we're at it, two more things (see below).

On Fri, 9 May 2003, Igor Pechtchanski wrote:

> FWIW, the previous patch failed because of a new flag (-Q) in CVS.

This has bitten me in the past, so I'm going to make a suggestion: why not
specify each flag in opts on a separate line, e.g.,

char *opts = "I:"

instead of

char *opts = "I:R:S:E:p:a:c:e:d:f:ou:w:t:s:y:0:1:2:ihv";

I know it doesn't look quite as compact (and consumes 2 extra bytes per
option -- the TAB and the NL), but it will ease patching considerably.

And another thing, I noticed there were some string constants in
cygrunsrv.h that had to be kept in sync with the numeric ones.  That kinda
rattled my sense of elegance (however much I have of it), so here's a
small patch (attached)...
2003-05-09  Igor Pechtchanski  <>

	* cygrunsrv.h: (STRINGIFY) New macro.
	(STRINGIFY_) New helper macro.
	(MAX_ENV_STR, MAX_DEPS_STR) Fix redundant string

      |\      _,,,---,,_
ZZZzz /,`.-'`'    -.  ;-;;,_
     |,4-  ) )-,_. ,\ (  `'-'		Igor Pechtchanski
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

Knowledge is an unending adventure at the edge of uncertainty.
  -- Leto II

Attachment: cygrunsrv-stringify.patch
Description: Text document

Unsubscribe info:
Problem reports:

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]