Shouldn't we update other with the value this returns?