bug#62412: 29.0.60; strange c++ indentation behavior with tree sitter

[Herman]

On 3/25/23 17:23, João Távora wrote:
> > I agree. In my opinion, c++-mode's heuristics are good.
> That's probably only because we're _used_ to c++-mode.  If we had been
> using c++-ts-mode for years, we would be equally suprised.
Yes, that can be true.

> > Tree-sitter support is new, it's expected that it won't work
> > perfectly. Also, it doesn't have to handle any invalid program. But,
> > while writing a program, it should handle indentation sensibly. I
> > don't think that it's a good approach that everybody who uses electric
> > indent should get used to the fact that whenever they writing a for
> > loop, the line will jump around. It's a bad experience.
> But writing a for loop from scratch is only one of the editing
> activities you do in a C++ file.  Other activities involve editing
> existing code.  In those situations, c++-ts-mode's heuristics could
> "win".  Unless you're willing to posit that writing code from scratch is
> more frequent than editing existing code, there's no right answer here.
What is the c++-ts heuristics here so it removes the indentation? I 
don't really understand why it does that. Is there a similarly looking 
situation where removing the indentation is the sensible behavior?


> > Anyways, feel free to close this issue if you think otherwise. I just
> > disabled ';'-caused auto indenting, so I don't see this unpleasant
> > behavior any more.
> Yes, i'm inclined to think that c++-ts-mode shouldn't add any chars to
> electric-indent-chars.  It's just not useful.
As far as I know, c++-mode has ';'-caused auto indenting, it just works 
with a different mechanism. So if the aim is that the two c++ modes 
should work similarly out of the box, then it'd make sense to keep 
electric-indent-chars as is.