Mailinglist Archive: zypp-devel (114 mails)
| < Previous | Next > |
Re: [zypp-devel] Re: [zypp-commit] r6311 - zypper-sources.cc - base urls iterator
- From: Duncan Mac-Vicar Prett <dmacvicar@xxxxxxx>
- Date: Tue, 31 Jul 2007 12:56:57 +0200
- Message-id: <200707311256.59777.dmacvicar@xxxxxxx>
On Tuesday 31 July 2007 09:18:44 Jan Kupec wrote:
> Let's keep baseUrls(), it's handy for the application to store the set
> of urls and then work with it if appropriate. Example use case: when
> initializing repos (checking if to refresh for those with autorefresh
> enabled), if the app finds out the repo is invalid, it can disable it in
> its internal set and ignore it for the rest of the operation.
>
> But for list repos operation, as is the case below, it is appropriate to
> use the iterator which always works with data of the *.repo files.
The problem is that with the variable replacement, the iterator type of
baseUrlsBegin() baseUrlsEnd() is different than std::set<Url>
Having the accessor to the baseUrls as a set with one iterator type and the
iterator itself would be confusing.
Michael?
Duncan
--
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx
> Let's keep baseUrls(), it's handy for the application to store the set
> of urls and then work with it if appropriate. Example use case: when
> initializing repos (checking if to refresh for those with autorefresh
> enabled), if the app finds out the repo is invalid, it can disable it in
> its internal set and ignore it for the rest of the operation.
>
> But for list repos operation, as is the case below, it is appropriate to
> use the iterator which always works with data of the *.repo files.
The problem is that with the variable replacement, the iterator type of
baseUrlsBegin() baseUrlsEnd() is different than std::set<Url>
Having the accessor to the baseUrls as a set with one iterator type and the
iterator itself would be confusing.
Michael?
Duncan
--
To unsubscribe, e-mail: zypp-devel+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: zypp-devel+help@xxxxxxxxxxxx
| < Previous | Next > |