Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources

2021-02-09 Thread Alexey Semenyuk
On Mon, 8 Feb 2021 22:26:22 GMT, Alexey Semenyuk wrote: > Remove needless `#ifdef __cplusplus` from .cpp sources > @alexeysemenyukoracle FYI, there was no need to force-push (or even push at > all) to your branch. It doesn't matter at all what the commit message(s) of > the commit(s) in the

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources

2021-02-09 Thread Kevin Rushforth
On Mon, 8 Feb 2021 22:26:22 GMT, Alexey Semenyuk wrote: > Remove needless `#ifdef __cplusplus` from .cpp sources @alexeysemenyukoracle FYI, there was no need to force-push (or even push at all) to your branch. It doesn't matter at all what the commit message(s) of the commit(s) in the source

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources [v2]

2021-02-09 Thread Ioi Lam
On Tue, 9 Feb 2021 15:46:55 GMT, Alexey Semenyuk wrote: >> Remove needless `#ifdef __cplusplus` from .cpp sources > > Alexey Semenyuk has refreshed the contents of this pull request, and previous > commits have been removed. The incremental views will show differences > compared to the

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources [v2]

2021-02-09 Thread Alexey Semenyuk
On Tue, 9 Feb 2021 04:03:14 GMT, Ioi Lam wrote: > The Bug Synopsis of 'Consider rewriting in C++ or moving to ".c" files' is > unclear. > I would request that the Bug Synopsis to be changed to "removed unnecessary > #ifdef __cplusplus" before integrating this PR. Bug Synopsis updated as

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources [v2]

2021-02-09 Thread Alexey Semenyuk
> Remove needless `#ifdef __cplusplus` from .cpp sources Alexey Semenyuk has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit