Re: RFR: 8286211: Update PCSC-Lite for Suse Linux to 1.9.5 [v3]

2022-05-23 Thread Weijun Wang
On Mon, 23 May 2022 21:44:39 GMT, Valerie Peng  wrote:

>> Need to update the 3 header files due to expiring business approval for 3rd 
>> party.
>> 
>> The header files contain tabs which jcheck disallows, so I have to replace 
>> them with spaces.
>> 
>> Thanks,
>> Valerie
>
> Valerie Peng has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Added a link for COPYING as on the pcsc-lite website

LGTM.

-

Marked as reviewed by weijun (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8853


Re: RFR: 8286211: Update PCSC-Lite for Suse Linux to 1.9.5 [v3]

2022-05-23 Thread Valerie Peng
> Need to update the 3 header files due to expiring business approval for 3rd 
> party.
> 
> The header files contain tabs which jcheck disallows, so I have to replace 
> them with spaces.
> 
> Thanks,
> Valerie

Valerie Peng has updated the pull request incrementally with one additional 
commit since the last revision:

  Added a link for COPYING as on the pcsc-lite website

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8853/files
  - new: https://git.openjdk.java.net/jdk/pull/8853/files/42c9df2a..32b75b6a

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=8853=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=8853=01-02

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8853.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8853/head:pull/8853

PR: https://git.openjdk.java.net/jdk/pull/8853


Re: RFR: 8286211: Update PCSC-Lite for Suse Linux to 1.9.5 [v2]

2022-05-23 Thread Valerie Peng
> Need to update the 3 header files due to expiring business approval for 3rd 
> party.
> 
> The header files contain tabs which jcheck disallows, so I have to replace 
> them with spaces.
> 
> Thanks,
> Valerie

Valerie Peng has updated the pull request incrementally with one additional 
commit since the last revision:

  Fix indentation

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8853/files
  - new: https://git.openjdk.java.net/jdk/pull/8853/files/8344ddb5..42c9df2a

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=8853=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=8853=00-01

  Stats: 192 lines in 3 files changed: 0 ins; 1 del; 191 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8853.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8853/head:pull/8853

PR: https://git.openjdk.java.net/jdk/pull/8853


RFR: 8286211: Update PCSC-Lite for Suse Linux to 1.9.5

2022-05-23 Thread Valerie Peng
Need to update the 3 header files due to expiring business approval for 3rd 
party.

The header files contain tabs which jcheck disallows, so I have to replace them 
with spaces.

Thanks,
Valerie

-

Commit messages:
 - Replaced tab with spaces in order to pass jcheck.
 - 8286211: Update PCSC-Lite for Suse Linux to 1.9.5

Changes: https://git.openjdk.java.net/jdk/pull/8853/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=8853=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8286211
  Stats: 221 lines in 4 files changed: 2 ins; 14 del; 205 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8853.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8853/head:pull/8853

PR: https://git.openjdk.java.net/jdk/pull/8853