percyashu commented on a change in pull request #830:
URL: https://github.com/apache/fineract/pull/830#discussion_r429565765
##########
File path:
fineract-provider/src/main/java/org/apache/fineract/interoperation/util/MathUtil.java
##########
@@ -87,6 +87,7 @@ public static Long abs(Long value) {
/** @return calculates minimum of the two values considering null values
* @param notNull if true then null parameter is omitted, otherwise
returns null */
+ @SuppressWarnings("NullTernary")
Review comment:
Will look more into those remaining two remaining errors and provide if
it is necessary to use the @SuppressWarnings.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
