[midPoint-git] [Evolveum/midpoint] f56b9d: Don't use `Throwable` in `catch` statement

Tony Tkáčik noreply at github.com
Fri Jan 24 15:20:49 CET 2025


  Branch: refs/heads/master
  Home:   https://github.com/Evolveum/midpoint
  Commit: f56b9d4f14d9160a78e5d5410c8f8fc41ed918f4
      https://github.com/Evolveum/midpoint/commit/f56b9d4f14d9160a78e5d5410c8f8fc41ed918f4
  Author: Andrej Zan <andrej.zan at evolveum.com>
  Date:   2025-01-24 (Fri, 24 Jan 2025)

  Changed paths:
    M repo/repo-cache/src/main/java/com/evolveum/midpoint/repo/cache/invalidation/ChangeDescription.java

  Log Message:
  -----------
  Don't use `Throwable` in `catch` statement

**Why**

Catching `Throwable` to "catch them all" is a bad practice, because it
catches also `Error`'s which in general, should not be catched, because
you "usuealy" can not do anything with them.


  Commit: bd3bf65f87a5dda3d7fc5b790bfc8e4e4ffd7146
      https://github.com/Evolveum/midpoint/commit/bd3bf65f87a5dda3d7fc5b790bfc8e4e4ffd7146
  Author: Tony Tkáčik <tonydamage at users.noreply.github.com>
  Date:   2025-01-24 (Fri, 24 Jan 2025)

  Changed paths:
    M repo/repo-cache/src/main/java/com/evolveum/midpoint/repo/cache/invalidation/ChangeDescription.java

  Log Message:
  -----------
  Merge pull request #258 from Evolveum/throwable-to-exception

Don't use `Throwable` in `catch` statement


Compare: https://github.com/Evolveum/midpoint/compare/2dc3bc0c6c18...bd3bf65f87a5

To unsubscribe from these emails, change your notification settings at https://github.com/Evolveum/midpoint/settings/notifications


More information about the midPoint-svn mailing list