MANIFEST | 2 button.c | 194 +++++++++++++++++++++++++---------------------- charproc.c | 26 +++--- debian/changelog | 22 +++++ fontutils.c | 29 ++----- graphics_regis.c | 65 ++++++++++----- menu.c | 28 ++++-- menu.h | 8 + misc.c | 18 ++++ package/debian/changelog | 6 + package/freebsd/Makefile | 2 package/xterm.spec | 4 ptyx.h | 23 +++++ screen.c | 6 - version.h | 6 - xterm.h | 9 +- xterm.log.html | 46 ++++++++++- xterm.man | 20 ++++ 18 files changed, 349 insertions(+), 165 deletions(-)
New commits: commit 632c582c411fd8a3513d0dd0781b4f02f3da35a2 Author: Sven Joachim <svenj...@gmx.de> Date: Fri Sep 19 18:21:44 2014 +0200 Close various bugs which are fixed according to upstream diff --git a/debian/changelog b/debian/changelog index 695d997..e2f8b2c 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,6 +1,22 @@ xterm (311-1) UNRELEASED; urgency=low * New upstream release. + - Mention xclip in manpage as an alternative workaround for copying + clipboard data, noting that selectToClipboard is the recommended + approach (Closes: #639094). + - Correct initialization for regular-expression feature of + exec-selectable and insert-selectable actions. + Modify logic for exec-formatted and insert-formatted actions to + ensure that the formatting occurs just after the selection is + received (Closes: #758633). + - Account for state of reverse-video in special case of cursor coloring + (Closes: #759734). + - Enable the "Escape Sequence" menu entry when an OSC 50 is received. + Also disable it if the escape sequence specifies no font + (Closes: #760208). + - Improve fix for the fontsel menu entry from xterm 304: because the + recovery used the "current font", it would fail if one first selected + a valid font, then an invalid font (Closes: #760207). -- Sven Joachim <svenj...@gmx.de> Fri, 19 Sep 2014 18:05:27 +0200 commit f761d93b18679f91c8eb00b1b3d423b3d2b79710 Author: Sven Joachim <svenj...@gmx.de> Date: Fri Sep 19 18:05:32 2014 +0200 New upstream release diff --git a/debian/changelog b/debian/changelog index 9f3079c..695d997 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +xterm (311-1) UNRELEASED; urgency=low + + * New upstream release. + + -- Sven Joachim <svenj...@gmx.de> Fri, 19 Sep 2014 18:05:27 +0200 + xterm (310-1) unstable; urgency=low * New upstream release. commit 2982e471d559c69d331ec2f8b4e26c5c583f76c0 Author: Sven Joachim <svenj...@gmx.de> Date: Fri Sep 19 18:03:56 2014 +0200 Imported Upstream version 311 diff --git a/MANIFEST b/MANIFEST index 22b4d14..634ec2c 100644 --- a/MANIFEST +++ b/MANIFEST @@ -1,4 +1,4 @@ -MANIFEST for xterm-310, version xterm-310 +MANIFEST for xterm-311, version xterm-311 -------------------------------------------------------------------------------- MANIFEST this file 256colres.h resource-definitions for 256-color mode diff --git a/button.c b/button.c index 69936a5..77929c9 100644 --- a/button.c +++ b/button.c @@ -1,4 +1,4 @@ -/* $XTermId: button.c,v 1.473 2014/05/26 17:12:51 tom Exp $ */ +/* $XTermId: button.c,v 1.481 2014/09/18 00:17:35 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -2177,16 +2177,39 @@ SelectionReceived(Widget w, #endif for (i = 0; i < text_list_count; i++) { size_t len = removeControls(xw, text_list[i]); + if (screen->selectToBuffer) { - size_t have = (screen->internal_select - ? strlen(screen->internal_select) + InternalSelect *mydata = &(screen->internal_select); + size_t have = (mydata->buffer + ? strlen(mydata->buffer) : 0); size_t need = have + len + 1; - char *buffer = realloc(screen->internal_select, need); + char *buffer = realloc(mydata->buffer, need); + + screen->selectToBuffer = False; +#if OPT_PASTE64 + screen->base64_paste = mydata->base64_paste; +#endif +#if OPT_READLINE + screen->paste_brackets = mydata->paste_brackets; +#endif if (buffer != 0) { strcpy(buffer + have, text_list[i]); - screen->internal_select = buffer; + mydata->buffer = buffer; } + TRACE(("FormatSelect %d.%d .. %d.%d %s\n", + screen->startSel.row, + screen->startSel.col, + screen->endSel.row, + screen->endSel.col, + mydata->buffer)); + mydata->format_select(w, mydata->format, mydata->buffer, + &(screen->startSel), + &(screen->endSel)); + + free(mydata->format); + free(mydata->buffer); + memset(mydata, 0, sizeof(*mydata)); } else { _WriteSelectionData(xw, (Char *) text_list[i], len); } @@ -3204,7 +3227,7 @@ do_select_regex(TScreen *screen, CELL *startc, CELL *endc) char *search; int *indexed; - TRACE(("Select_REGEX:%s\n", NonNull(expr))); + TRACE(("Select_REGEX[%d]:%s\n", inx, NonNull(expr))); if (okPosition(screen, &ld, startc) && expr != 0) { if (regcomp(&preg, expr, REG_EXTENDED) == 0) { int firstRow = firstRowOfLine(screen, startc->row, True); @@ -3234,7 +3257,7 @@ do_select_regex(TScreen *screen, CELL *startc, CELL *endc) int start_col = indexToCol(indexed, len, start_inx); int finis_col = indexToCol(indexed, len, finis_inx); - if (start_col <= actual && + if (start_col >= actual && actual < finis_col) { int test = finis_col - start_col; if (best_len < test) { @@ -4559,53 +4582,33 @@ getEventTime(XEvent *event) } /* obtain the selection string, passing the endpoints to caller's parameters */ -static char * -getSelectionString(XtermWidget xw, - Widget w, - XEvent *event, - String *params, - Cardinal *num_params, - CELL *start, CELL *finish) +static void +doSelectionFormat(XtermWidget xw, + Widget w, + XEvent *event, + String *params, + Cardinal *num_params, + FormatSelect format_select) { TScreen *screen = TScreenOf(xw); -#if OPT_PASTE64 - int base64_paste = (int) screen->base64_paste; -#endif -#if OPT_READLINE - int paste_brackets = (int) SCREEN_FLAG(screen, paste_brackets); -#endif + InternalSelect *mydata = &(screen->internal_select); + + memset(mydata, 0, sizeof(*mydata)); + mydata->format = x_strdup(params[0]); + mydata->format_select = format_select; /* override flags so that SelectionReceived only updates a buffer */ #if OPT_PASTE64 + mydata->base64_paste = screen->base64_paste; screen->base64_paste = 0; #endif #if OPT_READLINE + mydata->paste_brackets = screen->paste_brackets; SCREEN_FLAG_unset(screen, paste_brackets); #endif screen->selectToBuffer = True; - screen->internal_select = 0; xtermGetSelection(w, getEventTime(event), params + 1, *num_params - 1, NULL); - screen->selectToBuffer = False; - - if (screen->internal_select != 0) { - TRACE(("getSelectionString %d:%s\n", - (int) strlen(screen->internal_select), - screen->internal_select)); - *start = screen->startSel; - *finish = screen->endSel; - } else { - memset(start, 0, sizeof(*start)); - memset(finish, 0, sizeof(*finish)); - } -#if OPT_PASTE64 - screen->base64_paste = (Cardinal) base64_paste; -#endif -#if OPT_READLINE - if (paste_brackets) - SCREEN_FLAG_set(screen, paste_brackets); -#endif - return screen->internal_select; } /* obtain data from the screen, passing the endpoints to caller's parameters */ @@ -4625,10 +4628,13 @@ getDataFromScreen(XtermWidget xw, String method, CELL *start, CELL *finish) int save_firstValidRow = screen->firstValidRow; int save_lastValidRow = screen->lastValidRow; + const Cardinal noClick = 0; + int save_numberOfClicks = screen->numberOfClicks; + SelectUnit saveUnits = screen->selectUnit; - SelectUnit saveMap = screen->selectMap[0]; + SelectUnit saveMap = screen->selectMap[noClick]; #if OPT_SELECT_REGEX - char *saveExpr = screen->selectExpr[0]; + char *saveExpr = screen->selectExpr[noClick]; #endif Char *save_selection_data = screen->selection_data; @@ -4643,13 +4649,15 @@ getDataFromScreen(XtermWidget xw, String method, CELL *start, CELL *finish) screen->selection_size = 0; screen->selection_length = 0; - lookupSelectUnit(xw, 0, method); - screen->selectUnit = screen->selectMap[0]; + screen->numberOfClicks = 1; + lookupSelectUnit(xw, noClick, method); + screen->selectUnit = screen->selectMap[noClick]; memset(start, 0, sizeof(*start)); start->row = screen->cur_row; start->col = screen->cur_col; - *finish = *start; + finish->row = screen->cur_row; + finish->col = screen->max_col; ComputeSelect(xw, start, finish, False); SaltTextAway(xw, &(screen->startSel), &(screen->endSel)); @@ -4676,10 +4684,11 @@ getDataFromScreen(XtermWidget xw, String method, CELL *start, CELL *finish) screen->firstValidRow = save_firstValidRow; screen->lastValidRow = save_lastValidRow; + screen->numberOfClicks = save_numberOfClicks; screen->selectUnit = saveUnits; - screen->selectMap[0] = saveMap; + screen->selectMap[noClick] = saveMap; #if OPT_SELECT_REGEX - screen->selectExpr[0] = saveExpr; + screen->selectExpr[noClick] = saveExpr; #endif screen->selection_data = save_selection_data; @@ -4974,35 +4983,39 @@ freeArgv(char *blob, char **argv) } } +static void +reallyExecFormatted(Widget w, char *format, char *data, CELL *start, CELL *finish) +{ + XtermWidget xw; + + if ((xw = getXtermWidget(w)) != 0) { + char **argv; + char *blob; + int argc; + + if ((argv = tokenizeFormat(format)) != 0) { + blob = argv[0]; + for (argc = 0; argv[argc] != 0; ++argc) { + argv[argc] = expandFormat(xw, argv[argc], data, start, finish); + } + executeCommand(argv); + freeArgv(blob, argv); + } + } +} + void HandleExecFormatted(Widget w, - XEvent *event GCC_UNUSED, + XEvent *event, String *params, /* selections */ Cardinal *num_params) { XtermWidget xw; - if ((xw = getXtermWidget(w)) != 0) { - TRACE(("HandleExecFormatted(%d)\n", *num_params)); - - if (*num_params > 1) { - CELL start, finish; - char *data; - char **argv; - char *blob; - int argc; - - data = getSelectionString(xw, w, event, params, num_params, - &start, &finish); - if ((argv = tokenizeFormat(params[0])) != 0) { - blob = argv[0]; - for (argc = 0; argv[argc] != 0; ++argc) { - argv[argc] = expandFormat(xw, argv[argc], data, &start, &finish); - } - executeCommand(argv); - freeArgv(blob, argv); - } - } + TRACE(("HandleExecFormatted(%d)\n", *num_params)); + if ((xw = getXtermWidget(w)) != 0 && + (*num_params > 1)) { + doSelectionFormat(xw, w, event, params, num_params, reallyExecFormatted); } } @@ -5041,32 +5054,34 @@ HandleExecSelectable(Widget w, } } +static void +reallyInsertFormatted(Widget w, char *format, char *data, CELL *start, CELL *finish) +{ + XtermWidget xw; + + if ((xw = getXtermWidget(w)) != 0) { + char *exps; + + if ((exps = expandFormat(xw, format, data, start, finish)) != 0) { + unparseputs(xw, exps); + unparse_end(xw); + free(exps); + } + } +} + void HandleInsertFormatted(Widget w, - XEvent *event GCC_UNUSED, + XEvent *event, String *params, /* selections */ Cardinal *num_params) { XtermWidget xw; - if ((xw = getXtermWidget(w)) != 0) { - TRACE(("HandleInsertFormatted(%d)\n", *num_params)); - - if (*num_params > 1) { - CELL start, finish; - char *data; - char *temp = x_strdup(params[0]); - char *exps; - - data = getSelectionString(xw, w, event, params, num_params, - &start, &finish); - if ((exps = expandFormat(xw, temp, data, &start, &finish)) != 0) { - unparseputs(xw, exps); - free(exps); - } - free(data); - free(temp); - } + TRACE(("HandleInsertFormatted(%d)\n", *num_params)); + if ((xw = getXtermWidget(w)) != 0 && + (*num_params > 1)) { + doSelectionFormat(xw, w, event, params, num_params, reallyInsertFormatted); } } @@ -5092,6 +5107,7 @@ HandleInsertSelectable(Widget w, exps = expandFormat(xw, temp, data, &start, &finish); if (exps != 0) { unparseputs(xw, exps); + unparse_end(xw); free(exps); } free(data); diff --git a/charproc.c b/charproc.c index 4faed18..67d07e6 100644 --- a/charproc.c +++ b/charproc.c @@ -1,4 +1,4 @@ -/* $XTermId: charproc.c,v 1.1367 2014/07/12 22:49:54 Steve.Wall Exp $ */ +/* $XTermId: charproc.c,v 1.1370 2014/09/15 23:39:44 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -7972,8 +7972,8 @@ VTInitialize(Widget wrequest, DefaultFontNames[fWide] = x_strdup(wnew->misc.default_font.f_w); DefaultFontNames[fWBold] = x_strdup(wnew->misc.default_font.f_wb); #endif - TScreenOf(wnew)->MenuFontName(fontMenu_fontescape) = NULL; - TScreenOf(wnew)->MenuFontName(fontMenu_fontsel) = NULL; + TScreenOf(wnew)->EscapeFontName() = NULL; + TScreenOf(wnew)->SelectFontName() = NULL; TScreenOf(wnew)->menu_font_number = fontMenu_default; init_Sres(screen.initial_font); @@ -9830,7 +9830,9 @@ ShowCursor(void) } } } - if (T_COLOR(screen, TEXT_CURSOR) == xw->dft_foreground) { + if (T_COLOR(screen, TEXT_CURSOR) == (reversed + ? xw->dft_background + : xw->dft_foreground)) { setCgsBack(xw, currentWin, currentCgs, fg_pix); } setCgsFore(xw, currentWin, currentCgs, bg_pix); @@ -9906,7 +9908,9 @@ ShowCursor(void) * Set up a new request. */ if (filled) { - if (T_COLOR(screen, TEXT_CURSOR) == xw->dft_foreground) { + if (T_COLOR(screen, TEXT_CURSOR) == (reversed + ? xw->dft_background + : xw->dft_foreground)) { setCgsBack(xw, currentWin, currentCgs, fg_pix); } setCgsFore(xw, currentWin, currentCgs, bg_pix); @@ -10764,7 +10768,7 @@ DoSetSelectedFont(Widget w, } else { Boolean failed = False; int oldFont = TScreenOf(xw)->menu_font_number; - String save = TScreenOf(xw)->MenuFontName(fontMenu_fontsel); + String save = TScreenOf(xw)->SelectFontName(); char *val; char *test = 0; char *used = 0; @@ -10794,14 +10798,14 @@ DoSetSelectedFont(Widget w, && used != 0 && !strchr(used, '\n') && (test = x_strdup(used)) != 0) { - TScreenOf(xw)->MenuFontName(fontMenu_fontsel) = test; + TScreenOf(xw)->SelectFontName() = test; if (!xtermLoadFont(term, xtermFontName(used), True, fontMenu_fontsel)) { failed = True; free(test); - TScreenOf(xw)->MenuFontName(fontMenu_fontsel) = save; + TScreenOf(xw)->SelectFontName() = save; } } else { failed = True; @@ -10830,7 +10834,7 @@ FindFontSelection(XtermWidget xw, const char *atom_name, Bool justprobe) Atom target; if (!atom_name) - atom_name = (screen->mappedSelect + atom_name = ((screen->mappedSelect && atomCount) ? screen->mappedSelect[0] : "PRIMARY"); TRACE(("FindFontSelection(%s)\n", atom_name)); @@ -10850,10 +10854,10 @@ FindFontSelection(XtermWidget xw, const char *atom_name, Bool justprobe) target = XmuInternAtom(XtDisplay(xw), *pAtom); if (justprobe) { - screen->MenuFontName(fontMenu_fontsel) = + screen->SelectFontName() = XGetSelectionOwner(XtDisplay(xw), target) ? _Font_Selected_ : 0; TRACE(("...selected fontname '%s'\n", - NonNull(screen->MenuFontName(fontMenu_fontsel)))); + NonNull(screen->SelectFontName()))); } else { XtGetSelectionValue((Widget) xw, target, XA_STRING, DoSetSelectedFont, NULL, diff --git a/fontutils.c b/fontutils.c index 1221ef8..9456c43 100644 --- a/fontutils.c +++ b/fontutils.c @@ -1,4 +1,4 @@ -/* $XTermId: fontutils.c,v 1.440 2014/07/13 00:34:40 Ross.Combs Exp $ */ +/* $XTermId: fontutils.c,v 1.443 2014/09/03 23:58:53 tom Exp $ */ /* * Copyright 1998-2013,2014 by Thomas E. Dickey @@ -51,12 +51,6 @@ #include <stdio.h> #include <ctype.h> -#define ALLOC_STRING(name) \ - if (name != 0) \ - name = x_strdup(name) -#define FREE_STRING(name) \ - free_string(name) - #define SetFontWidth(screen,dst,src) (dst)->f_width = (src) #define SetFontHeight(screen,dst,src) (dst)->f_height = dimRound((screen)->scale_height * (float) (src)) @@ -126,12 +120,6 @@ typedef struct { char *end; } FontNameProperties; -static void -free_string(String value) -{ - free((void *) value); -} - #if OPT_RENDERFONT static void fillInFaceSize(XtermWidget, int); #endif @@ -1145,7 +1133,9 @@ xtermLoadFont(XtermWidget xw, &fnts[fNorm], warn[fNorm], (fontnum == fontMenu_default))) { - SetItemSensitivity(fontMenuEntries[fontnum].widget, False); + if (fontnum != fontMenu_fontsel) { + SetItemSensitivity(fontMenuEntries[fontnum].widget, False); + } goto bad; } @@ -1437,8 +1427,7 @@ xtermLoadFont(XtermWidget xw, FREE_STRING(screen->MenuFontName(fontnum)); screen->MenuFontName(fontnum) = tmpname; if (fontnum == fontMenu_fontescape) { - SetItemSensitivity(fontMenuEntries[fontMenu_fontescape].widget, - True); + update_font_escape(); } #if OPT_SHIFT_FONTS screen->menu_font_sizes[fontnum] = FontSize(fnts[fNorm].fs); @@ -1477,7 +1466,7 @@ xtermLoadFont(XtermWidget xw, free(tmpname); #if OPT_RENDERFONT - if (fontnum == fontMenu_fontsel) { + if ((fontnum == fontMenu_fontsel) && (fontnum != screen->menu_font_number)) { int old_fontnum = screen->menu_font_number; #if OPT_TOOLBAR SetItemSensitivity(fontMenuEntries[fontnum].widget, True); @@ -1492,8 +1481,10 @@ xtermLoadFont(XtermWidget xw, TRACE(("...recovering for TrueType fonts\n")); code = xtermLoadFont(xw, &myfonts, doresize, fontnum); if (code) { - SetItemSensitivity(fontMenuEntries[fontnum].widget, - UsingRenderFont(xw)); + if (fontnum != fontMenu_fontsel) { + SetItemSensitivity(fontMenuEntries[fontnum].widget, + UsingRenderFont(xw)); + } TRACE(("...recovered size %dx%d\n", FontHeight(screen), FontWidth(screen))); diff --git a/graphics_regis.c b/graphics_regis.c index d9a417d..7b02cd1 100644 --- a/graphics_regis.c +++ b/graphics_regis.c @@ -1,4 +1,4 @@ -/* $XTermId: graphics_regis.c,v 1.42 2014/07/13 09:10:24 tom Exp $ */ +/* $XTermId: graphics_regis.c,v 1.44 2014/09/17 08:35:49 tom Exp $ */ /* * Copyright 2014 by Ross Combs @@ -188,7 +188,7 @@ typedef struct RegisParseState { unsigned load_alphabet; unsigned load_w, load_h; unsigned load_index; - char load_glyph; + unsigned load_glyph; unsigned load_row; /* text options */ int string_rot_set; /* flag to distinguish string vs. character rotation */ @@ -3729,7 +3729,7 @@ parse_regis_command(RegisParseState *state) state->load_w = 8U; state->load_h = 10U; state->load_alphabet = 1U; - state->load_glyph = '\0'; + state->load_glyph = (unsigned) (unsigned char) '\0'; state->load_row = 0U; break; case 'P': @@ -4777,10 +4777,14 @@ parse_regis_option(RegisParseState *state, RegisGraphicsContext *context) (RegisterNum) register_num, r, g, b); continue; - } + } { + char skip; - TRACE(("DATA_ERROR: ignoring unexpected character in ReGIS screen color register mapping value \"%c\"\n", - pop_fragment(&optionarg))); + skip = pop_fragment(&optionarg); + (void) skip; /* variable needed only if tracing */ + TRACE(("DATA_ERROR: ignoring unexpected character in ReGIS screen color register mapping value \"%c\"\n", + skip)); + } return 1; } } @@ -5050,8 +5054,12 @@ parse_regis_option(RegisParseState *state, RegisGraphicsContext *context) } TRACE(("size multiplier: %s\n", fragment_to_tempstr(&sizemultiplierarg))); - load_regis_extent(fragment_to_tempstr(&sizemultiplierarg), - 0, 0, &ww, &hh); + if (!load_regis_extent(fragment_to_tempstr(&sizemultiplierarg), + 0, 0, &ww, &hh)) { + TRACE(("DATA_ERROR: unable to parse extent in '%c' command: \"%s\"\n", + state->option, fragment_to_tempstr(&sizemultiplierarg))); + break; + } if (!regis_num_to_int(&sizemultiplierarg, &sizemultiplier)) { TRACE(("DATA_ERROR: unable to parse extent in size multiplier option: \"%s\"\n", fragment_to_tempstr(&sizemultiplierarg))); @@ -5195,8 +5203,12 @@ parse_regis_option(RegisParseState *state, RegisGraphicsContext *context) } TRACE(("unitsize cell size: %s\n", fragment_to_tempstr(&unitsizearg))); - load_regis_extent(fragment_to_tempstr(&unitsizearg), 0, 0, - &unit_w, &unit_h); + if (!load_regis_extent(fragment_to_tempstr(&unitsizearg), 0, 0, + &unit_w, &unit_h)) { + TRACE(("DATA_ERROR: unable to parse extent in '%c' command: \"%s\"\n", + state->option, fragment_to_tempstr(&unitsizearg))); + break; + } if (!regis_num_to_int(&unitsizearg, &unitsize)) { TRACE(("DATA_ERROR: unable to parse extent in text unit cell size option: \"%s\"\n", fragment_to_tempstr(&unitsizearg))); @@ -5690,8 +5702,9 @@ parse_regis_items(RegisParseState *state, RegisGraphicsContext *context) if (extract_regis_string(input, state->temp, state->templen)) { switch (state->command) { case 'l': + /* FIXME: confirm that extra characters are ignored */ TRACE(("found character to load: \"%s\"\n", state->temp)); - state->load_glyph = state->temp[0]; /* FIXME: proper conversion */ + state->load_glyph = (unsigned) (unsigned char) state->temp[0]; state->load_row = 0U; break; case 't': @@ -5744,11 +5757,11 @@ parse_regis_items(RegisParseState *state, RegisGraphicsContext *context) unsigned glyph_size; val = strtoul(state->temp, NULL, 16); - TRACE(("found row %u for glyph \"%c\": \"%s\" value %02lx (%lu)\n", + TRACE(("found row %u for glyph %u: \"%s\" value %02lx (%lu)\n", state->load_row, state->load_glyph, state->temp, val, val)); if (state->load_row >= state->load_h) { - TRACE(("DATA_ERROR: ignoring extra glyph row for \"%c\"\n", + TRACE(("DATA_ERROR: ignoring extra row for glyph %u\n", state->load_glyph)); return 0; } @@ -5777,7 +5790,7 @@ parse_regis_items(RegisParseState *state, RegisGraphicsContext *context) unsigned unused_bits; glyph = &context->alphabets[state->load_index] - .bytes[(unsigned) state->load_glyph * glyph_size]; + .bytes[state->load_glyph * glyph_size]; bytew = GLYPH_WIDTH_BYTES(context->alphabets[state->load_index] .pixw); unused_bits = 8U - (context->alphabets[state->load_index].pixw @@ -5798,9 +5811,9 @@ parse_regis_items(RegisParseState *state, RegisGraphicsContext *context) state->load_row++; context->alphabets[state->load_index] - .loaded[(unsigned) state->load_glyph] = 1; + .loaded[state->load_glyph] = 1; #ifdef DEBUG_LOAD - TRACE(("marking alphabet %u at index %u glyph '%c' as loaded\n", + TRACE(("marking alphabet %u at index %u glyph %u as loaded\n", state->load_alphabet, state->load_index, state->load_glyph)); #endif @@ -5893,8 +5906,14 @@ parse_regis_toplevel(RegisParseState *state, RegisGraphicsContext *context) continue; if (state->input.pos >= state->input.len) break; - TRACE(("DATA_ERROR: skipping unknown token in optionset: \"%c\"\n", - pop_fragment(&state->input))); + { + char skip; + + skip = pop_fragment(&state->input); + (void) skip; /* variable needed only if tracing */ + TRACE(("DATA_ERROR: skipping unexpected character in optionset: \"%c\"\n", + skip)); + } /* FIXME: suboptions */ } state->option = '_'; @@ -5929,8 +5948,14 @@ parse_regis_toplevel(RegisParseState *state, RegisGraphicsContext *context) } if (state->input.pos >= state->input.len) return 0; - TRACE(("DATA_ERROR: skipping unknown token at top level: \"%c\"\n", - pop_fragment(&state->input))); + + { + char skip; + + skip = pop_fragment(&state->input); + (void) skip; /* variable needed only if tracing */ + TRACE(("DATA_ERROR: skipping unexpected character at top level: \"%c\"\n", ch)); + } return 0; } diff --git a/menu.c b/menu.c index d1c64ad..d4ae8d9 100644 --- a/menu.c +++ b/menu.c @@ -1,4 +1,4 @@ -/* $XTermId: menu.c,v 1.326 2014/07/12 22:50:28 Steve.Wall Exp $ */ +/* $XTermId: menu.c,v 1.327 2014/09/03 23:35:52 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -822,10 +822,7 @@ domenu(Widget w, if (IsEmpty(screen->menu_font_names[n][fNorm])) SetItemSensitivity(fontMenuEntries[n].widget, False); } - SetItemSensitivity( - fontMenuEntries[fontMenu_fontescape].widget, - (screen->menu_font_names[fontMenu_fontescape][fNorm] - ? True : False)); + update_font_escape(); update_menu_allowBoldFonts(); #if OPT_BOX_CHARS update_font_boxchars(); @@ -872,10 +869,10 @@ domenu(Widget w, SetItemSensitivity(fontMenuEntries[fontMenu_fontsel].widget, True); #else FindFontSelection(xw, NULL, True); - SetItemSensitivity( - fontMenuEntries[fontMenu_fontsel].widget, - (screen->menu_font_names[fontMenu_fontsel][fNorm] - ? True : False)); + SetItemSensitivity(fontMenuEntries[fontMenu_fontsel].widget, + (screen->SelectFontName() + ? True + : False)); #endif break; @@ -2633,6 +2630,19 @@ HandleAllowBoldFonts(Widget w, HANDLE_VT_TOGGLE(allowBoldFonts); } +#if OPT_LOAD_VTFONTS +void +update_font_escape(void) +{ + TScreen *screen = TScreenOf(term); + + SetItemSensitivity(fontMenuEntries[fontMenu_fontescape].widget, + ((screen->allowFontOps && + screen->EscapeFontName()) + ? True : False)); +} +#endif + #if OPT_DEC_CHRSET void HandleFontDoublesize(Widget w, diff --git a/menu.h b/menu.h index da937f4..e219415 100644 --- a/menu.h +++ b/menu.h @@ -1,4 +1,4 @@ -/* $XTermId: menu.h,v 1.133 2014/04/12 00:03:12 Ross.Combs Exp $ */ +/* $XTermId: menu.h,v 1.135 2014/09/04 00:17:55 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -452,6 +452,12 @@ extern void update_poponbell(void); #define update_marginbell() /* nothing */ +#if OPT_LOAD_VTFONTS +extern void update_font_escape(void); +#else +#define update_font_escape() /* nothing */ +#endif + #if OPT_ALLOW_XXX_OPS extern void update_menu_allowColorOps(void); extern void update_menu_allowFontOps(void); diff --git a/misc.c b/misc.c index d7872c6..bafe9e3 100644 --- a/misc.c +++ b/misc.c @@ -1,4 +1,4 @@ -/* $XTermId: misc.c,v 1.712 2014/05/26 14:45:58 tom Exp $ */ +/* $XTermId: misc.c,v 1.714 2014/09/03 23:59:25 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -3412,6 +3412,10 @@ ChangeFontRequest(XtermWidget xw, String buf) num = screen->menu_font_number; } name = x_strtrim(buf); + if (screen->EscapeFontName()) { + FREE_STRING(screen->EscapeFontName()); + screen->EscapeFontName() = 0; + } if (success && !IsEmpty(name)) { #if OPT_RENDERFONT if (UsingRenderFont(xw)) { @@ -3423,10 +3427,15 @@ ChangeFontRequest(XtermWidget xw, String buf) memset(&fonts, 0, sizeof(fonts)); fonts.f_n = name; SetVTFont(xw, num, True, &fonts); + if (num == screen->menu_font_number && + num != fontMenu_fontescape) { + screen->EscapeFontName() = x_strdup(name); + } } } else { Bell(xw, XkbBI_MinorError, 0); } + update_font_escape(); free(name); } } @@ -3528,6 +3537,7 @@ do_osc(XtermWidget xw, Char *oscbuf, size_t len, int final) * a special case. */ switch (mode) { + case 50: #if OPT_ISO_COLORS case OSC_Reset(4): case OSC_Reset(5): @@ -6044,3 +6054,9 @@ xtermEmbedWindow(Window winToEmbedInto) screen->embed_wide = (Dimension) attrs.width; } } + +void +free_string(String value) +{ + free((void *) value); +} diff --git a/package/debian/changelog b/package/debian/changelog index 2dd25ae..92638d6 100644 --- a/package/debian/changelog +++ b/package/debian/changelog @@ -1,3 +1,9 @@ +xterm-dev (311) unstable; urgency=low + + * maintenance updates + + -- Thomas E. Dickey <dic...@invisible-island.net> Mon, 28 Jul 2014 17:36:30 -0400 + xterm-dev (310) unstable; urgency=low * maintenance updates diff --git a/package/freebsd/Makefile b/package/freebsd/Makefile index 4a8e12a..9f4c4fc 100644 --- a/package/freebsd/Makefile +++ b/package/freebsd/Makefile @@ -5,7 +5,7 @@ # and "make makesum". PORTNAME= xterm -PORTVERSION= 310 +PORTVERSION= 311 CATEGORIES= x11 MASTER_SITES= ftp://invisible-island.net/xterm/ \ CRITICAL diff --git a/package/xterm.spec b/package/xterm.spec index 88c4089..c9a0fcc 100644 --- a/package/xterm.spec +++ b/package/xterm.spec @@ -1,11 +1,11 @@ -# $XTermId: xterm.spec,v 1.75 2014/07/14 23:36:58 tom Exp $ +# $XTermId: xterm.spec,v 1.76 2014/07/28 21:36:30 tom Exp $ Summary: X terminal emulator (development version) %global my_middle xterm %global my_suffix -dev %global fullname %{my_middle}%{my_suffix} %global my_class XTermDev Name: %{fullname} -Version: 310 +Version: 311 Release: 1 License: X11 Group: User Interface/X diff --git a/ptyx.h b/ptyx.h index 3b30255..d380455 100644 --- a/ptyx.h +++ b/ptyx.h @@ -1,4 +1,4 @@ -/* $XTermId: ptyx.h,v 1.809 2014/06/13 00:53:35 tom Exp $ */ +/* $XTermId: ptyx.h,v 1.813 2014/09/17 08:24:34 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -910,6 +910,23 @@ typedef enum { } TermColors; /* + * Definitions for exec-formatted and insert-formatted actions. + */ +typedef void (*FormatSelect) (Widget, char *, char *, CELL *, CELL *); + +typedef struct { + char *format; + char *buffer; + FormatSelect format_select; +#if OPT_PASTE64 + Cardinal base64_paste; +#endif +#if OPT_READLINE + unsigned paste_brackets; +#endif +} InternalSelect; + +/* * Constants for titleModes resource */ typedef enum { @@ -2294,7 +2311,7 @@ typedef struct { int lastValidRow; /* " " */ Boolean selectToBuffer; /* copy selection to buffer */ - char * internal_select; + InternalSelect internal_select; String default_string; String eightbit_select_types; @@ -2328,6 +2345,8 @@ typedef struct { String initial_font; String menu_font_names[NMENUFONTS][fMAX]; #define MenuFontName(n) menu_font_names[n][fNorm] +#define EscapeFontName() MenuFontName(fontMenu_fontescape) +#define SelectFontName() MenuFontName(fontMenu_fontsel) long menu_font_sizes[NMENUFONTS]; int menu_font_number; #if OPT_LOAD_VTFONTS || OPT_WIDE_CHARS diff --git a/screen.c b/screen.c index 0e2d241..fe06434 100644 --- a/screen.c +++ b/screen.c @@ -1,4 +1,4 @@ -/* $XTermId: screen.c,v 1.500 2014/06/19 21:09:11 tom Exp $ */ +/* $XTermId: screen.c,v 1.501 2014/07/30 08:06:23 tom Exp $ */ /* * Copyright 1999-2013,2014 by Thomas E. Dickey @@ -1254,8 +1254,8 @@ ScrnInsertChar(XtermWidget xw, unsigned n) if (col < first || col > last) { TRACE(("ScrnInsertChar - col %d outside [%d..%d]\n", col, first, last)); return; - } else if (last <= (col + (int) n)) { - n = (unsigned) (last - col); + } else if (last < (col + (int) n)) { + n = (unsigned) (last + 1 - col); } assert(screen->cur_col >= 0); diff --git a/version.h b/version.h index 10a1048..b7ef006 100644 --- a/version.h +++ b/version.h @@ -1,4 +1,4 @@ -/* $XTermId: version.h,v 1.395 2014/07/26 00:04:51 tom Exp $ */ +/* $XTermId: version.h,v 1.397 2014/09/18 21:12:19 tom Exp $ */ /* * Copyright 1998-2013,2014 by Thomas E. Dickey @@ -38,8 +38,8 @@ * version of X to which this version of xterm has been built. The resulting * number in parentheses is my patch number (Thomas E. Dickey). */ -#define XTERM_PATCH 310 -#define XTERM_DATE 2014-07-25 -- To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/e1xv0zg-0005ag...@moszumanska.debian.org