Re: [Fenfire-dev] Mudyc merge request

[Matti Katila]

On Mon, 6 Oct 2003, Tuomas Lukka wrote:
>       +++ mod/include/vob/vobs/Trivial.hxx
>       @@ -300,6 +300,39 @@
> 
>        VOB_DEFINED(TransTest);
> 
>        +
>        +/** A Vob to draw a selection with 3 'selection modes'.
>        + * 2nd coordinate system is used to select the current mode.
>        + * The mode is represented by a vob. These 3 possibile 'selection 
> mode'
>        + * vobs are set in parameters.
>        + * <p>
>        + * Modes(2nd cs square size width):
>        + *    <=1 normal,
>        + *    <= 2 pre selection and
>        + *    other is post selection.
>        + */
> 
> This is not good: the selectVob is a far more general tool,

"A vob which has three vobs and one of these three are rendered depending 
of the first coordinate system parameters." better?

If it's general, there must be possibility to give n vobs with variable 
size of cs parameters.

> you are now talking about a single application. 
> The javadoc should not mention anything like
> pre selection, post selection except as an *example* of how it can be used.

This is native code.
 
> Also, why is t0 the one to be passed to the vobs and t1 the condition?

curry is good, yes.

> What kind of a name is "FocusWithBuoysManager"? It doesn't tell me anything.

You haven't ever asked what does the BuoyViewMainNode or BuoyViewNodeType
tell me =)  So, after reading half an hour for BuoyView prefix it started 
to make sense and I took the [Multi]BuoyManager name to please more 
other developers ;)

> AbstractSelectVob should also be in lava for the time being, along with 
> SelectItemVob &c.

ok


   -Matti