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

RE: For the curious: Setup.exe char-> String patch




> -----Original Message-----
> From: Gary R Van Sickle [mailto:tiberius@braemarinc.com]
> 
> I think it would behoove us greatly to duplicate the semantics of
> std::string here, and return a zero-based offset on success, 
> and an "npos"
> on failure.
 
Whats a 'npos' defined as?
 
> > geturl.cc:
> >
> >  static void
> > -init_dialog (char const *url, int length, HWND owner)
> > +init_dialog (String const url, int length, HWND owner)
>                 ^^^^^^^^^^^^^^^^
> 
> This would be better written "const String &url".

Weeel, yes it would, yet another typo. Fortunately String copies are
very lightweight :}.
(One addition, one subtraction, and if test and 4 bytes of storage)

Rob


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