It looks completely OK but refactoring usually implies not changing the semantics. :)