[PATCH 2/5] d2d1: Partially implement RegisterEffectFromStream().

[Ziqing Hui]

On 6/7/22 11:43 AM, Ziqing Hui wrote:
> 
>>>> +    reg->info->default_input_count = input_count;
>>>> +    reg->info->min_inputs = input_count;
>>>> +    reg->info->max_inputs = input_count;
>>> Is this supposed to be subproperties of an Input? Maybe we should have basic property support first, and store it there. Is min/max/default expressible in effect xml? <Input> attributes maybe?
>>
>> It is likely that they can be expressed in xml as properties of <Effect> like "DisplayName"/"Author", tests are still needed. 
>> And according to the current test, if I don't explicitly express them in xml, they will be equal to the number of <Input> nodes inside <Inputs>.
>>
> 
> I did some tests today. I didn't find a way to express min/max/default input count in XML.
> 
> I tried these ways:
> 
> * Property of <Effect>:
> 
>     <Effect>
>         ...
>         <Property name='MinInputs' type='uint32' value='1'/>
>     </Effect>
> 
> * Sub property of <Inputs>:
> 
>     <Effect>
>         ...
>         <Inputs>
>             <Property name='Min' type='uint32' value='1'/>
>         </Inputs>
>     <Effect>
> 
> * Attribute of <Inputs>:
> 
>     <Effect>
>         ...
>         <Inputs min='1'/>
>     <Effect/>
> 
> Unfortunately, none of them work.
> 
> 

Some typo here in the mail... the second effect nodes are all </Effect>.