Looks okayish to me, but it really needs: a) a useful patch description b) patches that introduce actual usersAlso why does the function have a __ prefix in the name?