Re: r30184 - lyx-devel/trunk/src/insets

2009-06-20 Thread rgheck
Jean-Marc Lasgouttes wrote: Having looked at how these are used, of course you could change the signature of either, but they are used to do very different things, and they really make most sense as they are. I think. Changing the former would just force the odocstringstream::str() call to the

Re: r30184 - lyx-devel/trunk/src/insets

2009-06-20 Thread Jean-Marc Lasgouttes
Having looked at how these are used, of course you could change the signature of either, but they are used to do very different things, and they really make most sense as they are. I think. Changing the former would just force the odocstringstream::str() call to the client; changing the lat

Re: r30184 - lyx-devel/trunk/src/insets

2009-06-19 Thread rgheck
rgheck wrote: Jean-Marc Lasgouttes wrote: rgheck writes: So we have two different Inset*::getCaptionText methods with different signatures? This looks ugly, I'd rather standardize on one. I've changed the name to "getCaptionAsPlaintext()", which is what it does. And is it warranted to have

Re: r30184 - lyx-devel/trunk/src/insets

2009-06-19 Thread rgheck
Jean-Marc Lasgouttes wrote: rgheck writes: So we have two different Inset*::getCaptionText methods with different signatures? This looks ugly, I'd rather standardize on one. I've changed the name to "getCaptionAsPlaintext()", which is what it does. And is it warranted to ha

Re: r30184 - lyx-devel/trunk/src/insets

2009-06-19 Thread Jean-Marc Lasgouttes
rgheck writes: >> So we have two different Inset*::getCaptionText methods with different >> signatures? This looks ugly, I'd rather standardize on one. >> > I've changed the name to "getCaptionAsPlaintext()", which is what it does. And is it warranted to have one with strings and one with stre

Re: r30184 - lyx-devel/trunk/src/insets

2009-06-19 Thread rgheck
Jean-Marc Lasgouttes wrote: rgh...@lyx.org writes: +docstring InsetCollapsable::getCaptionText(OutputParams const & runparams) const Is there a reason why this is in InsetCollapsable and not InsetText? It would be a good thing to lean up in InsetCollapsable all the stuff that is not

Re: r30184 - lyx-devel/trunk/src/insets

2009-06-19 Thread Jean-Marc Lasgouttes
rgh...@lyx.org writes: > +docstring InsetCollapsable::getCaptionText(OutputParams const & runparams) > const Is there a reason why this is in InsetCollapsable and not InsetText? It would be a good thing to lean up in InsetCollapsable all the stuff that is not related to collapsableness. It (and