You changes its meaning, but the name remains the same. That make me really worry about future merges with upstream code. Because it will unlikely to generate any error message in merge process and might easily results in wrong logic. Please keep the way it originally is if there is no significant benefit gain.