This is the mail archive of the cygwin-apps@cygwin.com 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]

Curious code in Installer::installOneSource


I'm working in this area to implement the MD5-on-install feature. Is there
any reason why I should not simplify this:

char msg[64];
strcpy (msg, "Installing");
Progress.SetText1 (msg);
log (LOG_PLAIN, String (msg) + " " + source.Cached ());

To this:
char msg[] = "Installing";
Progress.SetText1 (msg);
log (LOG_PLAIN, String (msg) + " " + source.Cached ());

Or even, this:
Progress.SetText1 ("Installing");
log (LOG_PLAIN, String ("Installing ") + source.Cached ());

(There is no other use of the msg variable outside the context shown here.)


Max.


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