Re: [OpenJDK 2D-Dev] RFR: JDK-8035821: Move psfont properties files from src/share/classes to src/share/lib

2014-02-27 Thread Phil Race
Hi, Leaving aside jigsaw, and regardless of whether they are modifiable, the question as to whether at runtime they should live in rt.jar or jre/lib is a separate question than where they would live in the source tree and one does not necessarily follow the other. You seem to be touching on the

Re: [OpenJDK 2D-Dev] RFR: JDK-8035821: Move psfont properties files from src/share/classes to src/share/lib

2014-02-27 Thread Mandy Chung
Hi Phil, Is psfont*.properties* file more of a resource file? Is there any reason (or specification) that psfont*.properties has to be in JRE/lib directory? One potential reason might be the startup performance (warm start/cold start) to avoid opening a jar file? They don't look like a user-

Re: [OpenJDK 2D-Dev] RFR: JDK-8035821: Move psfont properties files from src/share/classes to src/share/lib

2014-02-27 Thread Phil Race
Magnus, I completely understand that the content of the files and the runtime behaviour aren't affected, but logically you'd want to put things together, not spread out across the source tree. Then its much easier for people to understand what uses what. If a jigsaw build somehow can't cope with

Re: [OpenJDK 2D-Dev] [9] Review request for 8029339 Custom MultiResolution image support on HiDPI displays

2014-02-27 Thread Alexander Scherbatiy
On 2/22/2014 3:54 AM, Jim Graham wrote: Hi Alexandr, On 2/18/14 7:33 AM, Alexander Scherbatiy wrote: Hi Jim, Let's divide the discussion into two part. 1. Where it is better to hold resolution variants? Putting resolution variants in Image class brings some questions like: - S

Re: [OpenJDK 2D-Dev] RFR: JDK-8035821: Move psfont properties files from src/share/classes to src/share/lib

2014-02-27 Thread Magnus Ihse Bursie
Phil, First of all, note that this is a change in build only. This change will not affect the source code in question, nor how it is executed. The properties files will end up in the very same location as a result of the build, wether with or without this patch. Hence, there is no need to run a