> -----Original Message-----
> From: Robert Collins [mailto:[EMAIL PROTECTED]]
> > -----Original Message-----
> > From: Gary R Van Sickle [mailto:[EMAIL PROTECTED]]
> > 
> > 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?
>  

"static const size_type npos = -1;"  It's a member of basic_string, so
something like this would do it:

class String
{
        static const long npos = -1;
};

and then you just have to compare to "String::npos" everywhere.

I don't know if initing to -1 is actually legal in the class declaration,
but you get the drift.

> > > 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)

??? You're doing some sort of ref counting?  Isn't that a bit ambitious for
our current needs?

-- 
Gary R. Van Sickle
Braemar Inc.
11481 Rupp Dr.
Burnsville, MN 55337

<<attachment: winmail.dat>>

Reply via email to