Fix Singleton typo and extend reflection handling to remaining implementations - #3203
Conversation
PR SummaryThis PR fixes a typo and extends reflection handling to several Singleton implementations. It adds exception handling to prevent instantiation via reflection in Changes
autogenerated by presubmit.ai |
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Commits Considered (2)
Files Processed (6)
- singleton/src/main/java/com/iluwatar/singleton/BillPughImplementation.java (1 hunk)
- singleton/src/main/java/com/iluwatar/singleton/InitializingOnDemandHolderIdiom.java (1 hunk)
- singleton/src/main/java/com/iluwatar/singleton/IvoryTower.java (1 hunk)
- singleton/src/test/java/com/iluwatar/singleton/EnumIvoryTowerTest.java (2 hunks)
- singleton/src/test/java/com/iluwatar/singleton/SingletonTest.java (2 hunks)
- singleton/src/test/java/com/iluwatar/singleton/ThreadSafeDoubleCheckLockingTest.java (2 hunks)
Actionable Comments (0)
Skipped Comments (3)
-
singleton/src/main/java/com/iluwatar/singleton/BillPughImplementation.java [42-46]
readability: "Remove redundant comment."
-
singleton/src/main/java/com/iluwatar/singleton/InitializingOnDemandHolderIdiom.java [45-49]
readability: "Remove redundant comment."
-
singleton/src/main/java/com/iluwatar/singleton/IvoryTower.java [35-39]
readability: "Remove redundant comment."
b8ea511 to
28bb7da
Compare
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Commits Considered (2)
Files Processed (6)
- singleton/src/main/java/com/iluwatar/singleton/BillPughImplementation.java (1 hunk)
- singleton/src/main/java/com/iluwatar/singleton/InitializingOnDemandHolderIdiom.java (1 hunk)
- singleton/src/main/java/com/iluwatar/singleton/IvoryTower.java (1 hunk)
- singleton/src/test/java/com/iluwatar/singleton/EnumIvoryTowerTest.java (2 hunks)
- singleton/src/test/java/com/iluwatar/singleton/SingletonTest.java (2 hunks)
- singleton/src/test/java/com/iluwatar/singleton/ThreadSafeDoubleCheckLockingTest.java (2 hunks)
Actionable Comments (0)
Skipped Comments (3)
-
singleton/src/main/java/com/iluwatar/singleton/BillPughImplementation.java [42-46]
readability: "Remove redundant comment."
-
singleton/src/main/java/com/iluwatar/singleton/InitializingOnDemandHolderIdiom.java [45-49]
readability: "Remove redundant comment."
-
singleton/src/main/java/com/iluwatar/singleton/IvoryTower.java [35-39]
readability: "Remove redundant comment."
|
|
Looks good! Thank you for the contribution 🎉 @all-contributors please add @smile-ab for code |
|
I've put up a pull request to add @smile-ab! 🎉 |




Description:
Fix typo and extend reflection handling for remaining implementations for the Singleton pattern
Issue:
#3202