rP22225 hurriedly fixed a crash, but made the JPC cache fail - though we are not using it and it might fail for other reasons still.
- Group Reviewers
Restricted Owners Package (Owns No Changed Paths)
- rP22301: JPS - make the JPC cache usable again (rP22225 fix)
What Stan said, unless the complete type is actually very long (not on desktop, I can't check).
I liked the extra line between lines 722 and 723 but at that point I'm just being annoying 😆
Apart from that this is good for me.
|722 ↗||(On Diff #8117)|
I actually think splitting them helps with readability, so I'd push for doing it anyway. But yes IIRC the issue still exists with 2013 and is fixed in 2015.