| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
using NULL makes it clear enough what's going on
|
|
|
|
|
| |
found with sparse
warning: Using plain integer as NULL pointer
|
|
|
|
|
| |
as per sparse warnings:
warning: non-ANSI function declaration of function '...'
|
|
|
|
|
| |
in UTF8 encodings it was used to store part of character
(c << 16), but now that character is ensured to be 32 bits, it's not needed
|
|
|
|
|
|
|
| |
introduce ARRAY_SIZE, like in Linux kernel
use explicit structs, if not using arrays
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>
|
|
|
|
| |
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the start of the scrollback buffer
This issue has been discussed before:
https://bbs.archlinux.org/viewtopic.php?id=108640
Copy mode and “hardcopy -h” always begin at the start of the scrollback buffer.
If a user sets a large scrollback limit with little output, then copy mode and
the file written by “hardcopy -h” will begin with many blank lines before the
first line of output.
The attached patch limits the scrollback buffer traversal to begin at the first
line of output, instead of the beginning of the scrollback buffer.
Also, code for moving to %age of buffer is changed to use float division so
that two different rep_cnt will not jump to the same location, except for
buffers less than 100 lines. Previously, the computed line number is rounded
down to the nearest 100th due to integer division.
Bug: 49377
|
| |
|
| |
|
|
|
|
|
|
|
| |
Currently, 0% and 1% points to the same location in the scrollback buffer due
to this bug.
Bug: 49336
|
| |
|
|
|
|
|
| |
now that _GNU_SOURCE definition is in config.h we want to make sure that
all headers that are included know about it
|
|
|
|
|
|
|
|
| |
Including the header in the source file guarantees matching signatures,
thus preventing subtle errors.
Self-contained headers document the required headers and makes using it
easier to use the header in new source files.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
clang analyzer
Value stored to 'rep_cnt' is never read
|
|
|
|
| |
modern compilers know better what they are doing
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
move forward declarations of functions from extern.h to their own header
files
fix inclusion option passed in Makefile
cosmetic stuff on some structs
|
| |
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: Amadeusz Sławiński <amade@asmblr.net>
|
| |
|
|
|
|
|
|
| |
removed #define SIMPLESCREEN and it's #defines
also removed #define DETACH and RXVT_SOC
all of this options are now default
|
| |
|
|
|
|
| |
inspired by https://github.com/wrp/wscreen/commit/a434e5d6126a8160b91118db32010476998e05d3
|
| |
|
|
|
|
|
|
|
|
|
| |
The fixes include:
* Add missing return values for some functions,
* Initialize some variables
* Fix data declaration to match struct's definition
* Appropriate casting where needed
* Forward declaration for some functions
|
|
|
|
|
|
| |
Thank you Curtis Brown for the patch.
This resolves bug #27052.
|
|
|
|
|
| |
Make sure the allocated memory is always freed. Also, do not reference freed memory.
Thank you valgrind.
|
|
|
|
|
|
|
|
| |
The problem would show up with splits (both vertical and
horizontal), because the layer wouldn't remember the position the
cursor was moved to, and when screen refreshes the regions (if
there's more than one), it takes the cursor to the old position. The
fix is to simply remember the cursor position in the layer.
|
|
|
|
|
|
|
|
|
| |
'%P' in caption string will set '%?' to true if the current region is
in copy/paste mode. So, if you want to simply change the color etc. of
your caption, prepend '%?%P%{XXX}%?' to your caption and you're set ('XXX'
stands for the usual attribute/color modifier). If, on the other hand,
you want to change the caption string in copy mode, change your caption
string to '%?%PCaption for copy mode%:Your usual caption%?'.
|
|
|
|
|
| |
With mousetrack on, it's now possible to leave a mark using a left-click in
copy mode, or scroll using the scroll wheel.
|
|
|
|
|
|
| |
Revamp the display list (in works). The future changes are expected to add full
mouse control, and perhaps some other utility functions (e.g. detaching a
display, changing permissions perhaps? etc.)
|
| |
|
| |
|
| |
|
|
|
|
| |
I decided to leave the doc/ directory alone for the moment.
|
| |
|
|
|
|
|
| |
Make sure there is some search to repeat when ;, is pressed in copy
mode.
|