aherbert commented on code in PR #117:
URL: https://github.com/apache/commons-numbers/pull/117#discussion_r923517093
##########
commons-numbers-complex/src/test/java/org/apache/commons/numbers/complex/CStandardTest.java:
##########
@@ -300,6 +300,103 @@ private static void assertConjugateEquality(Complex z,
}
}
+ /**
+ * Assert the operation on the complex number satisfies the conjugate
equality.
+ * Assert the operation on the complex number is exactly equal to the
operation on
Review Comment:
Move the new javadoc you added. The method should essentially be documented
as before. What you have done is add a second operation as an argument. This
should be documented, after the existing javadoc, as tested to exactly match
the result of the first operation. I said put it before the <h2> section as
that is a natural break in the javadoc. Below that is an explanation of
conjugates.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
