Re: RFR: 8321428: Deprecate for removal the package java.beans.beancontext [v5]

2024-05-17 Thread Alexey Ivanov
On Wed, 15 May 2024 17:39:16 GMT, Larry Cable  wrote:

>> the beancontext package was added (by me) in JDK 1.2 to provide 
>> JavaBeans(tm) with a containment and services hierarchy.
>> 
>> based upon concepts from OpenDoc, which was a popular component model at the 
>> time, the API pre-dated the addition of language features such as 
>> annotations, and the invention and adoption of programming design patterns 
>> such as "Inversion of Control"  and/or "Dependency Injection".
>> 
>> This API (package) has not evolved with either the language or current 
>> design trends, as such it is, at best, anachronistic, and is probably an 
>> anti-pattern that should be avoided.
>> 
>> This package is therefore deprecated **FOR REMOVAL IN AT FUTURE RELEASE**
>
> Larry Cable has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   removed trailing whitespace reported  by jcheck

Marked as reviewed by aivanov (Reviewer).

-

PR Review: https://git.openjdk.org/jdk/pull/18569#pullrequestreview-2063230682


Re: RFR: 8321428: Deprecate for removal the package java.beans.beancontext [v5]

2024-05-15 Thread Larry Cable
> the beancontext package was added (by me) in JDK 1.2 to provide JavaBeans(tm) 
> with a containment and services hierarchy.
> 
> based upon concepts from OpenDoc, which was a popular component model at the 
> time, the API pre-dated the addition of language features such as 
> annotations, and the invention and adoption of programming design patterns 
> such as "Inversion of Control"  and/or "Dependency Injection".
> 
> This API (package) has not evolved with either the language or current design 
> trends, as such it is, at best, anachronistic, and is probably an 
> anti-pattern that should be avoided.
> 
> This package is therefore deprecated **FOR REMOVAL IN AT FUTURE RELEASE**

Larry Cable has updated the pull request incrementally with one additional 
commit since the last revision:

  removed trailing whitespace reported  by jcheck

-

Changes:
  - all: https://git.openjdk.org/jdk/pull/18569/files
  - new: https://git.openjdk.org/jdk/pull/18569/files/86680a2c..255b17d0

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18569&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18569&range=03-04

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

PR: https://git.openjdk.org/jdk/pull/18569