On Mon, Sep 04, 2006 at 11:21:13AM +0200, Abdelrazak Younes wrote:
> Andre Poenitz wrote:
> >The attached patch replaces hull types like "simple", "eqnarray" with an
> >enum 'hullSimple', 'hullEqnArray' etc.
> >
> >Should save a few CPU cycles, be more typesafe and prevent a controversy
> >in which string format the type should be stored.
> 
> I think we should do the same for dialog names i.e. replace
> 
>   char const * const dialognames[] = {
>   "aboutlyx", "bibitem", "bibtex", "box", "branch", "changes",...

For this I'd actually prefer a factory that's initialized by static
singleton or such.

The reason that this approach in not taken with the different hulls is
the 'mutate' feature that basically needs to know all types at once.

The dialog, otoh, are pretty independent of each other.

Andre'

Reply via email to