Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8083559 | emb-9 | Stefan Johansson | P4 | Resolved | Fixed | team |
JDK-8063587 | 8u45 | Stefan Johansson | P4 | Resolved | Fixed | b01 |
JDK-8057924 | 8u40 | Stefan Johansson | P4 | Resolved | Fixed | b08 |
JDK-8070704 | emb-8u47 | Stefan Johansson | P4 | Resolved | Fixed | team |
Some of the new files add by refactoring the gc code have introduced includes that are not correctly sorted. The includes should be sorted alphabetically but always with precompiled.hpp first, the other exception is that all includes guarded by defines should be placed after all other includes. For example:
#include "precompiled.hpp"
#include "prims/jni.h"
#include "prims/whitebox.hpp"
#include "utilities/debug.hpp"
#if INCLUDE_ALL_GCS
#include "gc_implementation/g1/g1AllocationContext.hpp"
#include "gc_implementation/g1/g1CollectedHeap.inline.hpp"
#include "gc_implementation/g1/heapRegion.hpp"
#endif // INCLUDE_ALL_GCS
Some of the new files also need to have the copyright text altered to be correct.
#include "precompiled.hpp"
#include "prims/jni.h"
#include "prims/whitebox.hpp"
#include "utilities/debug.hpp"
#if INCLUDE_ALL_GCS
#include "gc_implementation/g1/g1AllocationContext.hpp"
#include "gc_implementation/g1/g1CollectedHeap.inline.hpp"
#include "gc_implementation/g1/heapRegion.hpp"
#endif // INCLUDE_ALL_GCS
Some of the new files also need to have the copyright text altered to be correct.
- backported by
-
JDK-8057924 Sort includes and verify copyright for new files
-
- Resolved
-
-
JDK-8063587 Sort includes and verify copyright for new files
-
- Resolved
-
-
JDK-8070704 Sort includes and verify copyright for new files
-
- Resolved
-
-
JDK-8083559 Sort includes and verify copyright for new files
-
- Resolved
-