Re: About final cfg.h (config layer 0 discussion)

"Andriy N. Gritsenko" <[email protected]>
Newsgroups gmane.comp.video.mplayer.g2.devel
Message-ID <[email protected]>
    Hi, Andriy N. Gritsenko!

Sometime (on Monday, May 26 at 12:41) I've written something...
>>> Another example for using of CONF_TYPE_GROUP:
>>> 
>>> (*) constant quantizer [   ]
>>> ( ) variable quantizer ----------------------------
>>> minimum quant [  ]   maximum quant [  ]
>>> ...................
>>> ---------------------------------------------------
>>> 
>>> so we have CONF_TYPE_SELECT with two choises:
>>> 1) CONF_TYPE_GROUP including 'vqscale' integer.
>>> 2) CONG_TYPE_GROUP including 'vqmin', 'vqmax' and other options.
>>> 
>>> When we select constant quantizer then second group will be disabled,
>>> when we select second group then we cannot change value of 'vqscale'.
>>> Config of vf_lavc must have hints for that. Simplest way for it is just
>>> have "transparent" CONG_TYPE_GROUP type. Did you understand?

>>and how would you define the above situation with your types?
>>i can't see how you GROUP helps here.

>For example, we assume that name of group is group selection flag. Then
>if we define all below we will have all hints to form box above but don't
>have any changes for CLI syntax:

[.......]

    I like to write here another version for the same. We'll define that
option->desc for CONF_TYPE_GROUP have two special meanings:
NULL - this is menu separator line
"*"  - this is list of special choises (boxes inside box)

Then we could:

config_t var_vq_group[]={
 {"vqmin", &vqmin, CONF_TYPE_INT, CONF_RANGE, 1, 31, NULL, "minimun quant"},
 {"vqmax", &vqmin, CONF_TYPE_INT, CONF_RANGE, 1, 31, NULL, "maximum quant"},
.........
 {NULL, NULL, 0, 0, 0, 0, NULL, NULL}
}

config_t vq_groups[]={
 {"vqscale", &vqscale, CONF_TYPE_INT, CONF_RANGE, 1, 31, NULL, "constant quantizer"},
 {"", NULL, CONF_TYPE_GROUP, 0, 0, 0, var_vq_group, "variable quantizer"},
 {NULL, NULL, 0, 0, 0, 0, NULL, NULL}
}

.....
{"", NULL, CONF_TYPE_GROUP, 0, 0, 0, vq_groups, "*"},
.....

It seems even simpler than previous version and allows even more that two
group choises (and still no changes for CLI syntax). :)

    With best wishes.
    Andriy.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.