This is OK. If the function ever gets more involve than this, it would be nice to only have the test bits ifdefed (for example, a function containing only the bits in _ clause).