> There was two problems:
>  - uri.getPath() was returning null for jar URI, since it is not a standard 
> URI and thus we did not detect file type based on extension.
>  - Our signature detection for MP3 had a bug and did not detect MP3 
> correctly. See comments in code.
>  
> Fixed by adding function to extract file name from jar URI and also signature 
> detection was fixed for MP3.

Alexander Matveev has updated the pull request incrementally with one 
additional commit since the last revision:

  8293971: Loading new Media from resources can sometimes fail when loading 
from FXML [v2]

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/902/files
  - new: https://git.openjdk.org/jfx/pull/902/files/6ec8b2e7..9830b77e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=902&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=902&range=00-01

  Stats: 4 lines in 2 files changed: 1 ins; 0 del; 3 mod
  Patch: https://git.openjdk.org/jfx/pull/902.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/902/head:pull/902

PR: https://git.openjdk.org/jfx/pull/902

Reply via email to