Manfred Hanke

Results 76 comments of Manfred Hanke

I've already seen this unfortunate confusion in https://github.com/TNG/ArchUnit/issues/1152#issuecomment-1674776006. Sorry for the inconvenience! [`public static DescribedPredicate and(DescribedPredicate

> The current situation is obviously broken. It's not that obvious to me: A usage like `DescribedPredicate.and(predicate1, predicate2)` works as intended, right? > The warning can be logged only at...

One effect of the major version upgrade without further configuration change is that the `lang` attribute of the user guide HTML is not set anymore: ```diff - + ```

This is actually expected behavior: ArchUnit does not (and in general cannot) track the type of all objects at runtime, and ```java void callDoSomething() { MyInterface myInterface = new MyClassA();...

The information about implementation of interfaces is [available in ArchUnit](https://javadoc.io/doc/com.tngtech.archunit/archunit/latest/com/tngtech/archunit/core/domain/JavaClass.html#getAllSubclasses()), but your question is about resolving runtime/dynamic polymorphism. While it may seem obvious that ```java MyInterface myInterface = new MyClassA();...

Spotbugs has been updated with https://github.com/TNG/ArchUnit/commit/7781b41a4c2406d27999d45c5eec936d83812435.

The `architecture.rules.OSGiConfigs` / `Services` / `Servlets` / `SlingModels` / `Utils` run in ≤ 11 ms each, which seems plausible (and IMO quite acceptable 🙂). `General` is the only exception with...

Thanks for raising the issue and providing the example project; that's very helpful! 💚 Some of the presumably missing violations are due to the fact that local variables are currently...

But I'm actually unsure whether ```java public void doD() { B b = new B(); try { b.doB(); } catch (Ex ex) { // dependency NOT detected log.info("Exception from ex",...

Yes, exactly: The domain object for the [`TryCatchBlock`](https://www.javadoc.io/doc/com.tngtech.archunit/archunit/latest/com/tngtech/archunit/core/domain/TryCatchBlock.html) is already available, but this information was not added to the [`JavaClassDependencies`](https://github.com/TNG/ArchUnit/blob/v1.0.1/archunit/src/main/java/com/tngtech/archunit/core/domain/JavaClassDependencies.java#L44-L53).