Changes between Initial Version and Version 1 of Ticket #51423, comment 96
- Timestamp:
- 04/19/2021 04:08:02 PM (3 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #51423, comment 96
initial v1 1 @SergeyBiryukov What do I think? 1 @SergeyBiryukov Great job running the performance benchmarks on the 2 approaches! 2 3 4 >What do I think? 2 5 3 6 Type casting code: it's more concise with less lines of code, but IMO less understandable for what happens and why with different values or lack of value. … … 5 8 Why? When I read it, my first thought is: why is it type casting and doing the processing when there are no emails to be processed. 6 9 7 I'd vote for the empty/is_array version for quick understanding and eliminating unnecessary processing.10 I'd vote for the empty/is_array version for quick understanding, eliminating unnecessary processing, and clear showing the processing paths for different values (which helps with building happy and unhappy path testing scenarios).