Re: RFR: 8324233: Update JPEG Image Decoding Software to 9f [v3]

2024-02-29 Thread Kevin Rushforth
On Thu, 29 Feb 2024 09:28:04 GMT, Jayathirth D V  wrote:

>> IJG has released latest version of libjpeg 9f and we need to update our 
>> version also match 9f changes.
>> IJG reference : https://www.ijg.org/
>> 
>> With updated changes both headless and headful tests are green on all 
>> platforms.
>> 
>> Also while updating to 9f noticed that many files don't have latest 
>> copyright year and comments from previous version updates like 9e, so 
>> updated that part also to match 9f version.
>
> Jayathirth D V has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update proper number for files

Marked as reviewed by kcr (Lead).

-

PR Review: https://git.openjdk.org/jfx/pull/1372#pullrequestreview-1908835099


Re: RFR: 8324233: Update JPEG Image Decoding Software to 9f [v3]

2024-02-29 Thread Ambarish Rapte
On Thu, 29 Feb 2024 09:28:04 GMT, Jayathirth D V  wrote:

>> IJG has released latest version of libjpeg 9f and we need to update our 
>> version also match 9f changes.
>> IJG reference : https://www.ijg.org/
>> 
>> With updated changes both headless and headful tests are green on all 
>> platforms.
>> 
>> Also while updating to 9f noticed that many files don't have latest 
>> copyright year and comments from previous version updates like 9e, so 
>> updated that part also to match 9f version.
>
> Jayathirth D V has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update proper number for files

Marked as reviewed by arapte (Reviewer).

-

PR Review: https://git.openjdk.org/jfx/pull/1372#pullrequestreview-1908412114


Re: RFR: 8324233: Update JPEG Image Decoding Software to 9f [v3]

2024-02-29 Thread Jayathirth D V
> IJG has released latest version of libjpeg 9f and we need to update our 
> version also match 9f changes.
> IJG reference : https://www.ijg.org/
> 
> With updated changes both headless and headful tests are green on all 
> platforms.
> 
> Also while updating to 9f noticed that many files don't have latest copyright 
> year and comments from previous version updates like 9e, so updated that part 
> also to match 9f version.

Jayathirth D V has updated the pull request incrementally with one additional 
commit since the last revision:

  Update proper number for files

-

Changes:
  - all: https://git.openjdk.org/jfx/pull/1372/files
  - new: https://git.openjdk.org/jfx/pull/1372/files/5df260c7..b43a51e4

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

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

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