The destroy function for shared bucket types should not interchange
the use of a local variable and the 'void *data' parameter.  It works,
since the two are equivalent, but it's confusing to look at.  Where
possible, the local variable is removed entirely.  Some bucket types
have to dereference the pointer to free private data, though, so in
those cases the local variable remains but it is now used consistently.

Submitted by:  Greg Stein


git-svn-id: https://svn.apache.org/repos/asf/apr/apr-util/trunk@58153 13f79535-47bb-0310-9956-ffa450edef68
3 files changed