Pavel Bludov

Results 26 comments of Pavel Bludov

@hcoles Does #860 fix this issue? Our build is green after upgrading Pitest to 1.6.3.

> and dumping the byte code for at least this sequence of statements? ## Java code ```java import java.util.List; import java.util.Set; import java.util.stream.Collectors; public class TestClass { public Set test1(List...

From the compiler's point of view, any modifier of a private class member has the same access. The difference will appear when the outer class is divided and there is...

> Ctor in nested classes should be allowed to have public scope. I want to say that both options have advantages and disadvantages. If we need to choose exactly one,...

How will this check handle long class/method names like [this](https://docs.jboss.org/hibernate/orm/4.1/javadocs/org/hibernate/service/jdbc/connections/spi/AbstractDataSourceBasedMultiTenantConnectionProviderImpl.html)? What about ```java class AbstractClassWithSomeLongNameThatCannotBeChangedDueToNamingPolicyImpl implements AbstractClassWithSomeLongNameThatCannotBeChangedDueToNamingPolicy { @Override public abstract AbstractClassWithSomeLongNameThatCannotBeChangedDueToNamingPolicyImpl someMethodWithLondNameThatCannotBeChangedToo(int firstParameter, int secondParameter) {} ``` }

The last one is dubious: ![изображение](https://user-images.githubusercontent.com/900805/178112176-aa0f1a28-3aac-4c89-901d-b9d194ab23cf.png) Maybe `'\n'` will be suitable here.

Long files list: https://gist.github.com/pbludov/fe3786436e177939e2041e3596626296 355 files > 120 lines 200 files > 200 lines We can start with a limit of 200 lines, and then reduce the limit to 120.

Both xdoc and Java source file should be changed. Please run `mvn clean verify` locally.

1. ~I'm going to ~~fix~~ remove `testLocaleIsSupported` to unblock contributors who use unsupported locales.~ done 2. We should verify supported locales an a special way. Maybe we need to build...

Same for `java.util.Locale`. This class has ho setters, please add it to the list of immutable classes.