Fix empty catch blocks - add explanatory comments

Co-authored-by: carstenartur <3164220+carstenartur@users.noreply.github.com>

Author: Copilot

org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/heapwalking/JavaWatchExpressionFilter.java

@@ -46,7 +46,9 @@ public boolean canCreateWatchExpression(IVariable variable) {
 			if (variable.getValue() instanceof JDIPlaceholderValue){
 				return false;
 			}
-		} catch (DebugException e) {}
+		} catch (DebugException e) {
+			// ignore - unable to get variable value
+		}
 		return true;
 	}
 

org.eclipse.jdt.launching/launching/org/eclipse/jdt/internal/launching/JavaLaunchableTester.java

@@ -181,8 +181,12 @@ private boolean hasMain(IJavaElement element) {
 				}
             }
 		}
-		catch (JavaModelException e) {}
-		catch (CoreException ce){}
+		catch (JavaModelException e) {
+			// ignore - element may not be accessible
+		}
+		catch (CoreException ce){
+			// ignore - element may not be accessible
+		}
 		return false;
 	}
 
@@ -278,7 +282,9 @@ private boolean hasMethod(IJavaElement element, Object[] args) {
 				}
 			}
 		}
-		catch (JavaModelException e) {}
+		catch (JavaModelException e) {
+			// ignore - method may not exist
+		}
 		return false;
 	}
 
@@ -322,8 +328,12 @@ private boolean hasTypeWithAnnotation(IJavaElement element, String annotationTyp
 				}
 			}
 		}
-		catch (JavaModelException e) {}
-		catch (InvalidInputException e) {}
+		catch (JavaModelException e) {
+			// ignore - type may not have annotation
+		}
+		catch (InvalidInputException e) {
+			// ignore - scanner may fail on invalid input
+		}
 		return false;
 	}
 
@@ -400,7 +410,9 @@ private boolean hasMethodWithAnnotation(IJavaElement element, Object[] args) {
 				}
 			}
 		} catch (JavaModelException e) {
+			// ignore - method may not have annotation
 		} catch (InvalidInputException e) {
+			// ignore - scanner may fail on invalid input
 		}
 		return false;
 	}
@@ -456,7 +468,10 @@ private boolean hasProjectNature(IJavaElement element, String ntype) {
     		}
 	    	return false;
         }
-    	catch (CoreException e) {return false;}
+    	catch (CoreException e) {
+			// ignore - element may not have the specified nature
+			return false;
+		}
     }
 
 	/**
@@ -477,7 +492,9 @@ private boolean hasSuperclass(IJavaElement element, String qname) {
 				}
 			}
 		}
-		catch(JavaModelException e) {}
+		catch(JavaModelException e) {
+			// ignore - type may not have superclass hierarchy
+		}
 		return false;
 	}
 
@@ -534,7 +551,9 @@ private boolean hasItemsOnBuildPath(IJavaProject project, Set<IJavaProject> sear
 	        		return true;
 	        	}
 	        }
-		} catch (JavaModelException e) {}
+		} catch (JavaModelException e) {
+			// ignore - classpath may not be accessible
+		}
 		return false;
 	}
 
@@ -556,7 +575,9 @@ private boolean implementsInterface(IJavaElement element, String qname) {
 				}
 			}
 		}
-		catch(JavaModelException e) {}
+		catch(JavaModelException e) {
+			// ignore - type may not have interface hierarchy
+		}
 		return false;
 	}
 

org.eclipse.jdt.launching/launching/org/eclipse/jdt/internal/launching/LaunchingPlugin.java

@@ -272,7 +272,9 @@ public void vmChanged(org.eclipse.jdt.launching.PropertyChangeEvent event) {
 								}
 							}
 						}
-					} catch (CoreException e) {}
+					} catch (CoreException e) {
+						// ignore - unable to update launch configuration
+					}
 				}
 			}
 		}

org.eclipse.jdt.launching/launching/org/eclipse/jdt/internal/launching/SocketListenConnectorProcess.java

@@ -133,7 +133,9 @@ public void done(IJobChangeEvent event) {
 				} else {
 					try{
 						terminate();
-					} catch (DebugException e){}
+					} catch (DebugException e){
+						// ignore - unable to terminate connection
+					}
 				}
 			}
 		});