Demo console filter behaves odd

[krumelmonster]

Version: 24bd33a
Branch: master

Back-end/Renderer/Compiler/OS

Back-ends: imgui_impl_opengl3.cpp; imgui_impl_glfw.cpp; GLEW
Compiler: gcc 9.2.1
Operating System: GNU/Linux

My Issue/Question:

I'm having a hard time getting a proper Logging/Console setup and the "Console" Demo kind of leaves puzzled.

Concerning the filter: My understanding was that you could apply multiple filters and separate them by a comma, however, to my surprise x,-1 and -1,x behave differenty. Is this a bug or desired behaviour?
I couldn't really find a specification on how those filters work and the code doesn't make it obvious to me either.

Screenshots/Video

Standalone, minimal, complete and verifiable example:

see imgui_demo.cpp:3802

[ocornut]

See #2435

[krumelmonster]

This pr is a year old and has no feedback. The filter functionality isn't really in the scope of a Bloat-free Graphical User interface and the code is hardly documented, readable, or fit for production software. Why not outsource such bits to a separate header where it can be made clear that it may have been abandoned and/or have a lower code quality then the rest of the library?

[ocornut]

This pr is a year old and has no feedback.

There are about 480 opens tasks/pr here and I am the only person reviewing things thoroughly.
As a rule of thumb, any PR that doesn't very carefully follow coding style is generally flagged in my mind as "this is likely to have other more meaningful design issues". But I still have to look at it, I will.

It hasn't been abandoned and although I agree that advanced applications will be likely to want to implement their own, more advanced solution, I don't agree that it's particularly lower code quality, unreadable etc. For the limited scope of what it provides, it is also perfectly fit for production.

FYI

to my surprise x,-1 and -1,x behave differenty

This was designed as such as I initially believed it was more flexible to handle it this way. I understand this can be causing confusion and not disagreeing this could be reworked. We can move this discussion to #2435.