about summary refs log tree commit diff stats
Commit message (Collapse)AuthorAgeFilesLines
* 3950Kartik K. Agaram2017-06-242-4/+86
| | | | | | Fix a regression caused by commit 3919. Thanks Juan Crispin Hernandez for running into this.
* 3949Kartik K. Agaram2017-06-241-0/+9
|
* 3948Kartik K. Agaram2017-06-242-0/+13
|
* 3947Kartik K. Agaram2017-06-241-3/+5
| | | | Fix CI.
* 3946Kartik K. Agaram2017-06-232-6/+6
| | | | Fix the failing scenario of commit 3944.
* 3945Kartik K. Agaram2017-06-231-33/+51
| | | | | Fix a discrepancy between fake screen and real terminal behavior. (See recent commits.)
* 3944Kartik K. Agaram2017-06-232-2/+50
| | | | | | | Reintroduce the failing test of commit 3938. It has two problems: a) it's failing, and b) it's not failing the same way as with a real screen.
* 3943Kartik K. Agaram2017-06-236-153/+6
| | | | Undo commit 3938 and almost everything after. Let's do this right.
* 3942Kartik K. Agaram2017-06-235-38/+106
| | | | | | | | No, my conclusion in the previous commit was wrong. When you print a character on the right margin, the cursor coordinates always wrap around to the left margin on the next row. It's just that if you're at the bottom of the screen, scrolling gives the impression that the row didn't change.
* 3941Kartik K. Agaram2017-06-235-14/+49
| | | | | | | | | | | | | | | | | Even though the bug of commit 3938 is now fixed, I'm still trying to track down why the failure looked different on the fake screen than on the real one. Snapshot as I try to track down the difference. One key lesson is that the approach of commit 3860 -- updating the cursor before rather than after printing each character -- turns out to be untenable. A sequence of `print` followed by `cursor-position` needs to behave the same as the real screen. But it's still not clear how the real screen. When you get to the end of a line the cursor position wraps after print to the left margin (column 0) on the next row. When you get to the bottom right the cursor position wraps to the *bottom left* margin. How the heck does it know to scroll on the next print, then? Is there some hidden state in the terminal?
* 3940Kartik K. Agaram2017-06-221-1/+1
|
* 3939Kartik K. Agaram2017-06-224-6/+16
|
* 3938Kartik K. Agaram2017-06-222-4/+64
| | | | | | | Fix an out-of-bounds write to the screen when sandboxes aligned just right. Thanks Ella Couch for reporting this issue.
* 3937Kartik K. Agaram2017-06-222-4/+23
| | | | | Fix screen-checking functions to handle fake screen after scrolling. I can't believe I forgot about this during commit 3882.
* 3936Kartik K. Agaram2017-06-221-4/+4
|
* 3935Kartik K. Agaram2017-06-227-9/+9
|
* 3934Kartik K. Agaram2017-06-2014-490/+510
|
* 3933Kartik K. Agaram2017-06-201-12/+12
|
* 3932Kartik K. Agaram2017-06-201-5/+25
| | | | | Some corrections and one bugfix to channels after reviewing their implementation with Caleb Couch.
* 3931Kartik K. Agaram2017-06-191-0/+11
|
* 3930 - experimental bytecode interpreterKartik K. Agaram2017-06-198-0/+1108
|
* 3929Kartik K. Agaram2017-06-192-1/+3
|
* 3928Kartik K. Agaram2017-06-196-283/+286
|
* 3927Kartik K. Agaram2017-06-1979-9209/+9912
|
* 3926Kartik K. Agaram2017-06-192-4/+114
| | | | | | Bugfix: when you hit `enter`, the cursor-row does not increment in *one* special situation: when the line wraps and the cursor is right at the start of one of the wrapped lines.
* 3925Kartik K. Agaram2017-06-192-0/+399
| | | | Bugfix: adjust row when hitting ctrl-u on wrapped lines.
* 3924Kartik K. Agaram2017-06-181-1/+1
|
* 3923Kartik K. Agaram2017-06-182-30/+157
| | | | | Bugfix: ctrl-a leaves things consistent in the presence of wrapped lines.
* 3922Kartik K. Agaram2017-06-174-11/+20
|
* 3921Kartik K. Agaram2017-06-165-7/+12
|
* 3920Kartik K. Agaram2017-06-161-28/+29
|
* 3919Kartik K. Agaram2017-06-163-8/+176
| | | | Bugfix: up-arrow in combination with wrapped lines.
* 3918Kartik K. Agaram2017-06-162-6/+94
| | | | Bugfix: handle wrapped lines when moving to end of line.
* 3917Kartik K. Agaram2017-06-163-16/+13
| | | | Redo commit 3905 to always shutdown cleanly on any error raised.
* 3916 - minimal prints when commenting linesKartik K. Agaram2017-06-162-28/+282
|
* 3915Kartik K. Agaram2017-06-161-4/+2
|
* 3914Kartik K. Agaram2017-06-161-8/+8
| | | | Remember that `before-previous-line` deals with wrapped screen lines.
* 3913Kartik K. Agaram2017-06-152-4/+90
| | | | Bugfix in ctrl-u.
* 3912Kartik K. Agaram2017-06-151-34/+44
|
* 3911Kartik K. Agaram2017-06-151-19/+3
|
* 3910Kartik K. Agaram2017-06-151-0/+1
|
* 3909Kartik K. Agaram2017-06-152-18/+22
| | | | | | | | | In tests where a text has the wrong length, properly show the text observed to help debug failures. We now also consistently say 'text' in Mu errors, never 'string'. Thanks Ella Couch for reporting this long-standing issue.
* 3908Kartik K. Agaram2017-06-151-1/+18
| | | | | Replace an assertion failure with an error message. Thanks Ella Couch for reporting this issue.
* 3907 - standardize test failure messagesKartik K. Agaram2017-06-155-102/+66
|
* 3906Kartik K. Agaram2017-06-102-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Yet another attempt at decomposing incremental edits in some clean way. The new idea now is that I need to only modify the screen using a restricted vocabulary of actions: render-all render-recipe-side render-sandbox-side render-recipe-errors render-line-from-cursor render-line-from-start erase-line-from-cursor render-character-at-cursor erase-character-at-cursor However, decomposing insert-at-cursor is challenging; how to manipulate cursor-row and cursor-column without also pretending to print to screen? Do I need to decompose `editor` into multiple containers so that I can keep cursor-row and cursor-column with screen modifications? Here's what `editor` looks like after all layers: container editor [ data:&:duplex-list:char top-of-screen:&:duplex-list:char bottom-of-screen:&:duplex-list:char before-cursor:&:duplex-list:char left:num right:num bottom:num cursor-row:num cursor-column:num indent?:bool undo:&:list:&:operation redo:&:list:&:operation ] It's not obvious that there's a clean way to split all these fields.
* 3905Kartik K. Agaram2017-06-107-24/+27
| | | | | | | Standardize exit paths. Most layers now don't need to know about termbox. We can't really use `assert` in console-mode apps; it can't just exit because we want to be able to check assertion failures in tests.
* 3904Kartik K. Agaram2017-06-099-50/+50
| | | | Standardize functions to put the main object being modified first.
* 3903 - minimal render when pressing 'tab'Kartik K. Agaram2017-06-0911-3686/+3902
| | | | Turns out all I had to do was reset `go-render?` to false.
* 3902 - drop redundant redraw of recipe side on F4Kartik K. Agaram2017-06-0924-3339/+3446
| | | | | | This change is interesting because I only updated one test to gain confidence that F4 will never redraw the recipe side. (Most of the changes are to explicitly render-all before each scenario.)
* 3901Kartik K. Agaram2017-06-0913-1255/+1237
|