> Is that `pdf_dealloc' on iterator->gl_iterator ok? I think that the
    > previous `gl_list_iterator_free' call is disposing the memory pointed
    > by iterator->gl_iterator.
    > 

   Well, gl_list_iterator() returns a structure and structures are assignable,
   so it may be that gl_list_free() frees gl_list_iterator_t _members_ but not 
the
   actual structure. If it does in a future update then gl_list_iterator() 
should
   return a pointer if any coherence is to be kept.

Yes. It makes sense. We allocated that pointer and we should free it
:)

Did you pass your little test case to the new implementation? If yes,
please commit it to CVS. 

Do not forget to write a ChangeLog entry and to update the AUTHORS
file if appropriate.

Thanks.


Reply via email to