[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Re: RFR: 8277868: Use Comparable.compare() instead of surrogate code [v2]
From:       Roger Riggs <rriggs () openjdk ! java ! net>
Date:       2021-11-30 21:06:09
Message-ID: UF7L_-UyjOPrjT0YxgBBcgzcqXnF3ecehk6GPVhQuMA=.d0969a24-13a7-4c26-aec0-2cfd7fdda0f4 () github ! com
[Download RAW message or body]

On Mon, 29 Nov 2021 08:18:47 GMT, Сергей Цыпанов <duke@openjdk.java.net> \
wrote:

> > Instead of something like
> > 
> > long x;
> > long y;
> > return (x < y) ? -1 : ((x == y) ? 0 : 1);
> > 
> > we can use `return Long.compare(x, y);`
> > 
> > All replacements are done with IDE.
> 
> Сергей Цыпанов has updated the pull request incrementally with one \
> additional commit since the last revision: 
> 8277868: Use Integer.signum() in BasicTableUI

The core-libs file changes look fine.
A 'client' reviewer should take a look too.

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

Marked as reviewed by rriggs (Reviewer).

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


[prev in list] [next in list] [prev in thread] [next in thread]