Andre Poenitz wrote:
> Have you considered changing that params2string interface to
> void InsetBranchMailer::params2string(InsetBranchParams const &,
>                                       ostream & data)
> {
>       data << name << ' ';
>       data << name_ << ' ';
>       params.write(data);
>       // Add all_branches parameter to data:
>       data << params.branchlist.allBranches() << "\n";
> }
> 
> (or when we are at it even
> 
> ostream & InsetBranchMailer::operator<<(ostream & os,
>                                         InsetBranchParams const &)
> {
>       os << name << ' ';
>       os << name_ << ' ';
>       params.write(os);
>       // Add all_branches parameter to data:
>       os << params.branchlist.allBranches() << "\n";
>       return os;
> }
> 
> one concept less to grasp and even better re-usable...)

Nice. However, let's try and get it working first.

-- 
Angus

Reply via email to