Details
-
Bug
-
Resolution: Fixed
-
P2
-
9
-
b156
Backports
| Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
|---|---|---|---|---|---|---|
| JDK-8174316 | 10 | Dmitry Fazunenko | P2 | Resolved | Fixed | b01 |
Description
The following tests:
hotspot/test/gc/g1/TestHumongousShrinkHeap.java
hotspot/test/gc/g1/TestShrinkDefragmentedHeap.java
hotspot/test/gc/parallel/TestDynShrinkHeap.java
implement the method humanReadableByteCount which body was copied from the Stack Overflow site: https://stackoverflow.com/a/3758880
It's just a few lines of code, but it could cause legal issues. The method should be either re-implemented or removed.
Besides the potential legal issues, duplicating a code is not a good practice.
hotspot/test/gc/g1/TestHumongousShrinkHeap.java
hotspot/test/gc/g1/TestShrinkDefragmentedHeap.java
hotspot/test/gc/parallel/TestDynShrinkHeap.java
implement the method humanReadableByteCount which body was copied from the Stack Overflow site: https://stackoverflow.com/a/3758880
It's just a few lines of code, but it could cause legal issues. The method should be either re-implemented or removed.
Besides the potential legal issues, duplicating a code is not a good practice.
Attachments
Issue Links
- backported by
-
JDK-8174316 Get rid of the humanReadableByteCount() method in openjdk/hotspot
-
- Resolved
-
- relates to
-
JDK-8170923 get rid of multiple implementations of MemoryUsagePrinter in gc tests
-
- Open
-