noinst_HEADER kaccelaction.h

[Ellis Whitehead]

On Wednesday 02 October 2002 20:08, Tim Jansen wrote:
> HI...
>
> The header kdelibs/kdecore/kaccelaction.h (which defines KAccelAction and
> KAccelActions) is in noinst_HEADERS. But several methods in KAccel use
> KAccelAction, and KAccel is a public class.
>
> Shouldnt kaccelaction.h be installed then? 

No.  Nobody has any need whatsoever for kaccelaction.h outside of kdelibs.  
Exposing it would be a seriously bad move for internal API flexibility.

> Public APIs that return private classes are quite confusing.

On the other hand, adding KAccelAction* to the public API would make it more 
convoluted.  Programmers should just ignore the given methods as @internal. 
;)

Cheers,
Ellis