Angus Leeming <[EMAIL PROTECTED]> writes:

| Andre Poenitz wrote:
>> Have you considered changing that params2string interface to
| ...
>> (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...)
>
| Let's think about this a little more, and let's illustrate this with 
| examples of the code in use in the core.

Do we really have to make this even more complex?

Notes:

        - boost::any is almost an perfect fit here
        - serilizatoin is added to boost soon
        - probably could we even use xtl (oh well)

-- 
        Lgb

Reply via email to