Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes leahneukirchen/mblaze#194 #223

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbauhardt
Copy link

@mbauhardt mbauhardt commented Mar 22, 2022

fixes #194

Why
Configure the number of lines which are shown on top of mless would be nice to have.

How
Introduce property Scan-Lines available in $MBLAZE/profile to configure the number of lines to scan on top of mless.
mless reads this property and using it. if not set the default value of 6 is used to not break the existing behaviour.
Also updating the man pages for mless and mblaze-profile.

@leahneukirchen
Copy link
Owner

I don't think this will work, as we had the explicit cases exactly for 6 lines by default, you'd need to generalize the algorithm.

Also note that in POSIX sh you can use $((math)) instead of $(expr math).

@mbauhardt
Copy link
Author

I don't think this will work, as we had the explicit cases exactly for 6 lines by default, you'd need to generalize the algorithm.

ah. ok. i see. you mean we have to do a loop around https://github.com/leahneukirchen/mblaze/pull/223/files#diff-0f549fd15ac39a405676bb080ab4f082f510f787923824ac807f07b45b86a8e3R30 ?

this make the thing a bit more complicated. i will take a closer look later this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configure number of shown entries on mless ?
2 participants