msoucy / dncurses Goto Github PK

Functions that currently have an additional mv- function can have those rolled in with the originals via overloading.

As it is now, the logic for getstr() with no limit is kind of iffy. It should be verified and fixed as much as possible. Currently it doesn't handle ^X, M-X, or tabs properly.

Window-creating feels really complex and unintuitive. There should be a way to make it simpler without losing support for different situations

The user needs a way to disable automatically calling refresh() whenever put is called - in fact, this should probably be the default behavior.

Refresh might be better off calling wnoutrefresh and then doupdate later on...but not sure on that one.

If a program is built with rdmd, then (unless panel is imported) -lpanel isn't needed. However, with DUB, all source files are imported by default. As a result, -lpanel is needed when importing as a DUB package. Which is "fine", but I'd prefer not to link EVERYTHING and then tell it to strip unused stuff, since that messes with the result too much for my liking.

It's possible to prefix it with version(metus_dncurses_panel) and have users specify them in their "dflags", but the problem with this is that non-DUB users would need to manually add "-L-lpanel -version=metus_dncurses_panel", which doesn't really look nice, and is different depending on dmd/ldc/gdc anyways.

Alternatively, something like this inside panel.d:

version(Have_dncurses) {} // Defined by dub
else version=metus_dncurses_panel

Since this is meant to be a higher-level wrapper, it doesn't make sense to have a lot of the stuff that's in here. Figure out what names can be abstracted away.

Some functions for consideration:

• intrflush vs qiflush
• The four different input modes