Re: Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-10 Thread Jan-Marek Glogowski
On 10.01.22 11:22, Stephan Bergmann wrote: On 09/01/2022 09:41, Julien Nabet wrote: So it means we could at least use of compbase.hxx internally, then put compbase.hxx in deprecated. We can deprecate/discourage its internal use, but IMO should not deprecate it for external use (where we offer

Re: Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-10 Thread Stephan Bergmann
On 09/01/2022 09:41, Julien Nabet wrote: So it means we could at least use of compbase.hxx internally, then put compbase.hxx in deprecated. We can deprecate/discourage its internal use, but IMO should not deprecate it for external use (where we offer no alternative solution). I don't think w

Re: Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-09 Thread Julien Nabet
On 09/01/2022 18:12, Thorsten Behrens wrote: Julien Nabet wrote: So it means we could at least use of compbase.hxx internally, then put compbase.hxx in deprecated. Yep. ... Ok now I saw for example compbase11.hxx wasn't used (or it seemed so considering the only results of the git grep):

Re: Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-09 Thread Thorsten Behrens
Julien Nabet wrote: > So it means we could at least use of compbase.hxx internally, then put > compbase.hxx in deprecated. > Yep. Cheers, -- Thorsten signature.asc Description: PGP signature

Re: Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-09 Thread Julien Nabet
On 09/01/2022 07:27, Noel Grandin wrote: Those are from the days before we had reliable variable-parameters-templates. We can't remove them because they are part of the published API. Thank you for your feedback. So it means we could at least use of compbase.hxx internally, then put compb

Re: Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-08 Thread Noel Grandin
Those are from the days before we had reliable variable-parameters-templates. We can't remove them because they are part of the published API.

Question about cppuhelper/compbase and other compbase1/2/.../12

2022-01-08 Thread Julien Nabet
Hello, I noticed include/cppuhelper had files like: compbase1.hxx compbase2.hxx ... compbase12.hxx It seems the only diff is the number of parameters. But there's also compbase.hxx which can take any number of parameters. May we replace all the use of compabase.hxx by compbase.hxx or is t