Duplicated functions?

Hi all,

I've been digging around trying to track down my cursor issues, and I noticed
that there seem to be 3 identical implementation of the function
find_entry_by_id implemented in dlls/user/exticon.c, dlls/version/resource.c,
and loader/pe_resource.c.

Is this intentional?  It seems like the danger for the the functions to get out
of sync is quite high (especially since there aren't sufficient comments
indicating that other versions need to be updated).

If it isn't intentional, I'll open a bug about it (I don't really have time to
fix it right now), otherwise I'll add a couple of lines of documentation
indicating that there are multiple versions that need to be updated to each of
the functions comments.

Warren

[Alexandre Julliard]

[e-mail] writes:
> Is this intentional?  It seems like the danger for the the functions to get out
> of sync is quite high (especially since there aren't sufficient comments
> indicating that other versions need to be updated).

This is intentional, and it's unlikely these functions are going to
change. It will probably be fixed at some point, but there's no need
to worry about it for now.