Re: oafd patch I just commited

[Maciej Stachowiak <mjs eazel com>]

Jaka Mocnik <jaka mocnik kiss uni-lj si> writes:

> it is rather obvious why it was needed, I pressume:
>

Good catch. I guess this was never noticed because no one was calling
get_directories.

 - Maciej
 
> Index: oafd/ac-corba.c
> ===================================================================
> RCS file: /cvs/gnome/oaf/oafd/ac-corba.c,v
> retrieving revision 1.33
> diff -u -p -r1.33 ac-corba.c
> --- oafd/ac-corba.c	2000/10/06 04:25:33	1.33
> +++ oafd/ac-corba.c	2000/10/20 10:12:55
> @@ -386,7 +386,7 @@ impl_OAF_ActivationContext__get_director
>  	retval->_buffer =
>  		CORBA_sequence_OAF_ObjectDirectory_allocbuf (retval->_length);
>  
> -	for (i = 0, cur = servant->dirs; cur; cur = cur->next) {
> +	for (i = 0, cur = servant->dirs; cur; cur = cur->next, i++) {
>  		ChildODInfo *child;
>  		child = cur->data;
>  		retval->_buffer[i] = CORBA_Object_duplicate (child->obj, ev);
> Index: ChangeLog
> ===================================================================
> RCS file: /cvs/gnome/oaf/ChangeLog,v
> retrieving revision 1.104
> diff -u -p -r1.104 ChangeLog
> --- ChangeLog	2000/10/19 23:31:57	1.104
> +++ ChangeLog	2000/10/20 10:12:56
> @@ -1,3 +1,8 @@
> +2000-10-20  Jaka Mocnik  <jaka mocnik kiss uni-lj si>
> +
> +	* oafd/ac-corba.c (impl_OAF_ActivationContext__get_directories):
> +	increase i in the for loop.
> +
>  2000-10-19  Maciej Stachowiak  <mjs eazel com>
>  
>  	* NEWS, configure.in: Updated for 0.6
> 
> 
> -- 
> 
> email: jaka mocnik kiss uni-lj si
> w3:    http://pluton.ijs.si/~jaka
> 
> _______________________________________________
> gnome-components-list mailing list
> gnome-components-list gnome org
> http://mail.gnome.org/mailman/listinfo/gnome-components-list