<div class="gmail_quote">On Mon, Feb 1, 2010 at 3:47 PM, Jakob Leben <span dir="ltr"><<a href="mailto:jakob.leben@gmail.com">jakob.leben@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="gmail_quote"><div class="im">On Mon, Feb 1, 2010 at 12:36 PM, Laurent Aimar <span dir="ltr"><<a href="mailto:fenrir@via.ecp.fr" target="_blank">fenrir@via.ecp.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Hi,<br>
<br>
On Mon, Feb 01, 2010, Jakob Leben wrote:<br>
>    Hi all!<br>
><br>
> +void input_item_BeginAddSubItems( input_item_t *p_item )<br>
> +{<br>
> +  vlc_event_t event;<br>
> +  event.type = vlc_InputItemBeginAddSubItems;<br>
> +  vlc_event_send( &p_item->event_manager, &event );<br>
> +}<br>
> +<br>
> +void input_item_EndAddSubItems( input_item_t *p_item )<br>
> +{<br>
> +  vlc_event_t event;<br>
> +  event.type = vlc_InputItemEndAddSubItems;<br>
> +  vlc_event_send( &p_item->event_manager, &event );<br>
> +}<br>
 Why are they needed ?<br>
 They are a call to future bugs (way too easy to forget them<br>
or to break when moving arround code).<br></blockquote></div><div>
<br>Another idea was to have an event that reports for all the new children together, so modules should create an array and pass that in the event.<br></div></div></blockquote><div><br> I have second thoughts about that other idea: an event about all the parent nodes should be sent before their children, otherwise it doesn't make sense to notify about children of a parent about which itself you haven't notified yet.<br>
<br>But this to be possible will result in huge refactoring of some modules ( at least for the XSPF for sure ), and that is probably gonna bring much more bugs than the simple rules about Begin / Add / End .<br></div></div>