Re: RFR: 8280840: Update libFFI to 3.4.2 [v2]

2022-03-07 Thread Joeri Sykora 
On Thu, 24 Feb 2022 03:07:57 GMT, Alexander Matveev  
wrote:

>> LibFFI updated to 3.4.2. No additional changes to our code, libffi code or 
>> build system were required. Tested on all platforms.
>
> Alexander Matveev has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8280840: Update libFFI to 3.4.2 [v3]

Built and tested.

-

Marked as reviewed by sykora (Author).

PR: https://git.openjdk.java.net/jfx/pull/738

Re: RFR: 8280840: Update libFFI to 3.4.2 [v2]

2022-03-05 Thread Kevin Rushforth 
On Thu, 24 Feb 2022 03:07:57 GMT, Alexander Matveev  
wrote:

>> LibFFI updated to 3.4.2. No additional changes to our code, libffi code or 
>> build system were required. Tested on all platforms.
>
> Alexander Matveev has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8280840: Update libFFI to 3.4.2 [v3]

Looks good. Pending a second reviewer.

-

Marked as reviewed by kcr (Lead).

PR: https://git.openjdk.java.net/jfx/pull/738

Re: RFR: 8280840: Update libFFI to 3.4.2

2022-02-23 Thread Alexander Matveev 
On Sat, 19 Feb 2022 07:45:43 GMT, Alexander Matveev  
wrote:

> LibFFI updated to 3.4.2. No additional changes to our code, libffi code or 
> build system were required. Tested on all platforms.

[v2] - Fixed TAB issue.
[v3] - Fixed space alignment issue in aarch64/ffitarget.h.

-

PR: https://git.openjdk.java.net/jfx/pull/738

Re: RFR: 8280840: Update libFFI to 3.4.2 [v2]

2022-02-23 Thread Alexander Matveev 
> LibFFI updated to 3.4.2. No additional changes to our code, libffi code or 
> build system were required. Tested on all platforms.

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

  8280840: Update libFFI to 3.4.2 [v3]

-

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/738/files
  - new: https://git.openjdk.java.net/jfx/pull/738/files/2ca3a52a..a06a3e65

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jfx&pr=738&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=738&range=00-01

  Stats: 9 lines in 1 file changed: 0 ins; 0 del; 9 mod
  Patch: https://git.openjdk.java.net/jfx/pull/738.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/738/head:pull/738

PR: https://git.openjdk.java.net/jfx/pull/738