naver / arcus-java-client

ARCUS Java client
Apache License 2.0
50 stars 47 forks source link

INTERNAL: Suppress-warning for `this-escape` #827

Closed cheesecrust closed 1 month ago

cheesecrust commented 1 month ago

πŸ”— Related Issue

⌨️ What I did

ArcusClient 의 κ²½μš°μ—λŠ” μœ„ 처럼 λ©”μ„œλ“œλ₯Ό λ”°λ‘œ ν˜ΈμΆœν•΄μ•Ό ν•˜λŠ” 경우 λ‹€λ₯Έ ν•„λ“œλ“€λ„ public 으둜 μ ‘κ·Όμ œμ–΄λ₯Ό λ³€κ²½ν•΄μ•Όν•©λ‹ˆλ‹€. MemcachedClient 의 κ²½μš°μ—λŠ” 상속을 ν•˜λŠ” ν΄λž˜μŠ€κ°€ λ§Žμ•„μ„œ 이λ₯Ό μœ„μ™€ 같은 방식, λ©”μ„œλ“œλ₯Ό λ–Όμ–΄λ‚΄λŠ” λ°©μ‹μœΌλ‘œ μˆ˜μ •ν•˜κ²Œ 되면 μˆ˜μ • 사항이 λ§Žμ•„μ§‘λ‹ˆλ‹€. λ˜ν•œ MemcachedClient λŠ” ν˜„μž¬ 상속을 ν•˜κ³  있기 λ•Œλ¬Έμ— final class λ₯Ό μ μš©ν•  수 μ—†μŠ΅λ‹ˆλ‹€. ArcusClient λ˜ν•œ 상속 κ°€λŠ₯성이 μžˆλ‹€κ³  말씀해 μ£Όμ…”μ„œ λͺ¨λ‘ suppress-warning 을 μ μš©ν•˜μ˜€μŠ΅λ‹ˆλ‹€.