We have few cleanup tasks to be done.
1. Add error checks/variable validation
2. Remove any un-required/stale comments
3. Add comments wherever necessary.
4. Review TODOs and address any important EA blockers
5. Fix warnings
6. Java coding guidelines
7, Objective coding guidelines
8. Remove dead code
9. Error fallback/ no service / no memory /
We can do this file level, by creating a new sub task for set of files as and when we fix.
1. Add error checks/variable validation
2. Remove any un-required/stale comments
3. Add comments wherever necessary.
4. Review TODOs and address any important EA blockers
5. Fix warnings
6. Java coding guidelines
7, Objective coding guidelines
8. Remove dead code
9. Error fallback/ no service / no memory /
We can do this file level, by creating a new sub task for set of files as and when we fix.
- blocks
-
JDK-8271024 ☂ JavaFX : Implement Metal Rendering Pipeline
-
- Open
-
1.
|
MSLBackend: Remove NSLog |
|
Resolved | Ambarish Rapte | |
2.
|
ClearRect: Separate method for clearing rect |
|
Resolved | Ambarish Rapte | |
3.
|
Remove few trivial warnings |
|
Resolved | Ambarish Rapte | |
4.
|
warning: [removal] AccessController in java.security has been deprecated |
|
Resolved | Ambarish Rapte | |
5.
|
Remove unrequired MTLCommandBuffer handlers |
|
Resolved | Ambarish Rapte | |
6.
|
Revert JDK-8342522 |
|
Closed | Ambarish Rapte | |
7.
|
Few minor cleanups |
|
Resolved | Ambarish Rapte | |
8.
|
Refactor texture classes |
|
Resolved | Ambarish Rapte | |
9.
|
Remove return statement from JNI methods |
|
Resolved | Ambarish Rapte | |
10.
|
Remove optional native debug logs |
|
Resolved | Ambarish Rapte | |
11.
|
Remove near empty native files |
|
Open | Ambarish Rapte | |
12.
|
Cleanup JNI methods |
|
Open | Ambarish Rapte | |
13.
|
Change standard output prints to standard error |
|
Open | Ambarish Rapte | |
14.
|
Move metal shaders to msl directory |
|
Open | Ambarish Rapte | |
15.
|
Several cleanups in MTL Java files |
|
Open | Ambarish Rapte | |
16.
|
Remove MetalResourceFactory interface |
|
Open | Ambarish Rapte |