Piotr Idzik

Results 122 comments of Piotr Idzik

> > > > > Please `don't close`. > > > > > > > > > > > > Or merge. > > > > > > > >...

> > > > > > Please `don't close`. > > > > > > > > > > > > > > > Or merge. > > > >...

> Additionally, I think `LocalFinalVariableName` is wrongly marked and checked as `LocalVariableName`. @cpu-pixel Thanks for spotting that!

> `WhitespaceAround` it conflicts with `clang-format`. When I create the spaces clang-format undoes it and causes an error in the checkstyle. Please have a look at #5241. But you are...

I do not see much sense of these changes. @BamaCharanChhandogi could you please have a look?

Why did you close #703? This PR seems to be exactly the same as #703.

> #703 It's not clear why the other tests didn't pass, I didn't change any of the other code. It is just some _random failure_ - some tests in this...

In any case, the question about `aabb` from #703 remains valid.

The code looks quite good. Please update the _doc string_ by adding some explanation how do you find the longest palindrome, adding a sentence about _non-unique_ solution (I guess, your...

@changweidalaifu6 after rethinking the whole thing, I am not sure anymore if this contribution is suitable for this repository. The problem is very specific (it is a _leetcode_ problem after...