amorynan commented on code in PR #24241: URL: https://github.com/apache/doris/pull/24241#discussion_r1322762770
########## be/src/vec/data_types/data_type_decimal.h: ########## @@ -387,9 +387,9 @@ constexpr bool IsDataTypeDecimalOrNumber = template <typename FromDataType, typename ToDataType> requires IsDataTypeDecimal<FromDataType> && IsDataTypeDecimal<ToDataType> -ToDataType::FieldType convert_decimals(const typename FromDataType::FieldType& value, - UInt32 scale_from, UInt32 scale_to, - UInt8* overflow_flag = nullptr) { +typename ToDataType::FieldType convert_decimals(const typename FromDataType::FieldType& value, Review Comment: clion compiler auto optimize , I will drop this -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org