summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2010-06-21 04:00:16 +0200
committerDenys Vlasenko2010-06-21 04:00:16 +0200
commit05a550b48abe8490264cacf14b68534c2f8ac3ed (patch)
tree9613d48ed2d301ed2b80ff5f942a95557b7cadc9
parenta8b594fda7cbfcc2aadf9779ee40b264eeec86b2 (diff)
downloadbusybox-05a550b48abe8490264cacf14b68534c2f8ac3ed.zip
busybox-05a550b48abe8490264cacf14b68534c2f8ac3ed.tar.gz
fix errors in detecting non-modified areas
function old new delta screen_char 100 102 +2 cleanup 84 86 +2 screen_dump 215 210 -5 conspy_main 1503 1491 -12 Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--miscutils/conspy.c49
1 files changed, 25 insertions, 24 deletions
diff --git a/miscutils/conspy.c b/miscutils/conspy.c
index 7ba7959..ae471a3 100644
--- a/miscutils/conspy.c
+++ b/miscutils/conspy.c
@@ -62,7 +62,6 @@ struct globals {
int kbd_fd;
unsigned width;
unsigned height;
- char mask;
char last_attr;
struct screen_info info;
struct termios term_orig;
@@ -73,6 +72,17 @@ struct globals {
SET_PTR_TO_GLOBALS(xzalloc(sizeof(G))); \
} while (0)
+enum {
+ FLAG_v, // view only
+ FLAG_c, // create device if need
+ FLAG_s, // session
+ FLAG_n, // no colors
+ FLAG_d, // dump screen
+ FLAG_f, // follow cursor
+};
+#define FLAG(x) (1 << FLAG_##x)
+#define BW (option_mask32 & FLAG(n))
+
static void screen_read_close(int fd, char *data)
{
unsigned i, j;
@@ -95,7 +105,7 @@ static void screen_read_close(int fd, char *data)
static void screen_char(char *data)
{
- if (((G.last_attr - ATTR(data)) & G.mask) != 0) {
+ if (!BW && G.last_attr != ATTR(data)) {
// BLGCRMOW
static const char color[8] = "04261537";
@@ -123,29 +133,32 @@ static void gotoxy(int row, int line)
static void screen_dump(char *data)
{
- int space, linefeed, line, row;
+ int linefeed_cnt;
+ int line, row;
int linecnt = G.info.lines - G.y;
data += 2 * G.y * G.info.rows;
- for (linefeed = line = 0; line < linecnt && line < G.height; line++) {
- for (space = row = 0; row < G.info.rows; row++, NEXT(data)) {
+ linefeed_cnt = 0;
+ for (line = 0; line < linecnt && line < G.height; line++) {
+ int space_cnt = 0;
+ for (row = 0; row < G.info.rows; row++, NEXT(data)) {
unsigned tty_row = row - G.x; // if will catch row < G.x too
if (tty_row >= G.width)
continue;
- space++;
- if (((G.last_attr - ATTR(data)) & G.mask) && CHAR(data) == ' ')
+ space_cnt++;
+ if (BW && (CHAR(data) | ' ') == ' ')
continue;
- while (linefeed != 0) {
+ while (linefeed_cnt != 0) {
bb_putchar('\r');
bb_putchar('\n');
- linefeed--;
+ linefeed_cnt--;
}
- while (--space)
+ while (--space_cnt)
bb_putchar(' ');
screen_char(data);
}
- linefeed++;
+ linefeed_cnt++;
}
}
@@ -173,7 +186,7 @@ static void cleanup(int code)
close(G.kbd_fd);
}
// Reset attributes
- if (G.mask != 0)
+ if (!BW)
printf("\033[0m");
bb_putchar('\n');
if (code > 1)
@@ -234,16 +247,6 @@ static NOINLINE void start_shell_in_child(const char* tty_name)
}
}
-enum {
- FLAG_v, // view only
- FLAG_c, // create device if need
- FLAG_s, // session
- FLAG_n, // no colors
- FLAG_d, // dump screen
- FLAG_f, // follow cursor
-};
-#define FLAG(x) (1 << FLAG_##x)
-
int conspy_main(int argc UNUSED_PARAM, char **argv) MAIN_EXTERNALLY_VISIBLE;
int conspy_main(int argc UNUSED_PARAM, char **argv)
{
@@ -284,8 +287,6 @@ int conspy_main(int argc UNUSED_PARAM, char **argv)
sprintf(vcsa_name + sizeof("/dev/vcsa")-1, "%u", ttynum);
}
sprintf(tty_name, "%s%u", "/dev/tty", ttynum);
- if (!(opts & FLAG(n)))
- G.mask = 0xff;
if (opts & FLAG(c)) {
if ((opts & (FLAG(s)|FLAG(v))) != FLAG(v))
create_cdev_if_doesnt_exist(tty_name, makedev(4, ttynum));