Re: RFC: deb/ddeb support for debuginfod

2019-12-01 Thread Florian Weimer
* Florian Weimer: > * Frank Ch. Eigler: > >> +inline bool >> +string_endswith(const string& haystack, const string& needle) >> +{ >> + return (haystack.size() >= needle.size() && >> + haystack.substr(haystack.size()-needle.size()) == needle); >> +} > > I think you should use std::equal becau

Re: RFC: deb/ddeb support for debuginfod

2019-12-01 Thread Florian Weimer
* Frank Ch. Eigler: > +inline bool > +string_endswith(const string& haystack, const string& needle) > +{ > + return (haystack.size() >= needle.size() && > + haystack.substr(haystack.size()-needle.size()) == needle); > +} I think you should use std::equal because substr allocations. So som