See my comment on the tracker
As far as I know, it shouldn't break anything, and it has apparently worked for every 1G user that tried it.
However the 1G and 2G share a build, and only some 1Gs need the patch. I doubt many other 1G users and virtually no 2G users will have tried the patch (as they have no need!).
My suggestion would be put it in SVN prior to 3.2, to 'force' all 1G and 2G users to test it. If it screws up, revert prior to 3.2.