Commit Graph

441 Commits

Author SHA1 Message Date
Kartik K. Agaram 26640c9101 new test
For commit e4e12c77ad which fixed a regression caused by commit
24a0d162ef.
2022-06-12 10:59:42 -07:00
Kartik K. Agaram 4f64784833 new test
For commit ff88a2a927 which fixed a regression caused by commit
e51ce12969.

I'm trying to provide enough guardrails for myself and future readers
without causing a combinatorial explosion in tests. The previous commit
was able to get more value out of existing tests, but this test feels
necessary. And useful in general without reference to a specific bug.
2022-06-12 10:34:53 -07:00
Kartik K. Agaram 059efba79d override mouse state lookups in tests
If I'd had this stuff in my test harness earlier, two recent commits
would have failed tests and given me early warning:
  ff88238ff1
  ff88a2a927
2022-06-12 09:04:21 -07:00
Kartik K. Agaram 9ada7cf833 delete all tests once they've executed 2022-06-12 07:37:02 -07:00
Kartik K. Agaram e8ff9f285f redundant check 2022-06-12 07:35:52 -07:00
Kartik K. Agaram e4e12c77ad fix a second BSOD in #4 :/
I need more tests.
2022-06-12 07:20:14 -07:00
Kartik K. Agaram ff88a2a927 bugfix in commit e51ce12969
Any time I press a ctrl- chord LÖVE actually sees two key chords:
  C-lctrl
  C-... (the real one)

But it's not just that. There's also a lot in the codebase that's just
habit-based. I need more tests.
2022-06-12 07:03:35 -07:00
Kartik K. Agaram ff88238ff1 bugfix: BSOD in #4.
I messed up a function call in commit 391d764e13.
2022-06-11 20:54:17 -07:00
Kartik K. Agaram e51ce12969 bugfix: autosave and undo in a couple of cases
This fixes part of #4, but not the BSOD.
2022-06-11 20:47:04 -07:00
Kartik K. Agaram 24a0d162ef bugfix: missed fixing a callsite 2022-06-11 20:41:51 -07:00
Kartik K. Agaram f8cdd01661 . 2022-06-11 16:07:34 -07:00
Kartik K. Agaram 66b17d798d things seem to feel snappier now
However, I think a lot of the benefit comes from just turning JIT off.
Turning it on is still noticably sluggish.
2022-06-10 15:08:51 -07:00
Kartik K. Agaram 391d764e13 stop handling nil screen_line_starting_pos everywhere
Things seem to be working..
2022-06-10 15:07:11 -07:00
Kartik K. Agaram e7787b979f this fixes the immediate regression 2022-06-10 14:52:49 -07:00
Kartik K. Agaram 0dd940024e stop repeatedly checking for line wrapping
We have a regression since we started reclaiming love Text fragments
more aggressively in commit 69c5d844cc. Pressing pageup no longer knows
about any line's screen lines. Not fixed yet.
2022-06-10 14:38:39 -07:00
Kartik K. Agaram d202e2ce99 slight reorg in Readme 2022-06-10 14:19:27 -07:00
Kartik K. Agaram 299890f593 more precise scroll on paste 2022-06-10 14:19:02 -07:00
Kartik K. Agaram feedc51227 faster paste
We don't need to perform the scroll calculations after inserting every
single character from the clipboard.
2022-06-10 14:19:02 -07:00
Kartik K. Agaram 41521518a3 revert previous commit 2022-06-10 13:59:39 -07:00
Kartik K. Agaram 019a3b48f1 experiment: extremely precise scrolling on paste
But this is too slow.
2022-06-10 13:59:15 -07:00
Kartik K. Agaram 1c56b9d644 clean up memory leak experiments 2022-06-10 13:46:59 -07:00
Kartik K. Agaram 88209ec343 while we're at it, undo naming points 2022-06-10 13:45:50 -07:00
Kartik K. Agaram 242a087016 set current_drawing_index with current_drawing 2022-06-10 13:44:12 -07:00
Kartik K. Agaram fcfe667fe9 include drawing index in a few places 2022-06-10 13:40:07 -07:00
Kartik K. Agaram c93553b670 . 2022-06-10 12:01:46 -07:00
Kartik K. Agaram c875f7be46 stop saving the entire file when modifying drawings
Now we just disallow that entirely.
2022-06-10 11:48:32 -07:00
Kartik K. Agaram 79a1241370 undo creating new drawings 2022-06-10 11:45:53 -07:00
Kartik K. Agaram fcacb6e63d extract scrolling logic out of insert_at_cursor 2022-06-10 11:29:01 -07:00
Kartik K. Agaram 31418976d4 extract scrolling logic out of insert_return 2022-06-10 11:21:41 -07:00
Kartik K. Agaram 69c5d844cc remove some memory leaks from rendered fragments
All signs so far seem to be that CPU is cheap for this application, but
memory is expensive. It's easy to get sluggish if the GC comes on.

After some experiments using https://github.com/yaukeywang/LuaMemorySnapshotDump,
one source of memory leaks is rendered fragments (https://love2d.org/wiki/Text
objects). I need to render text in approximately word-sized fragments to
mostly break lines more intelligently at word boundaries.

I've attached the files I used for my experiments (suffixed with a '.')

There's definitely still a leak in fragments. The longer I edit, the
more memory goes to them.
2022-06-10 11:16:41 -07:00
Kartik K. Agaram fbad3dd205 avoid some string concatenations
file:write can write multiple args one after another; no need to
concatenate them first.

I'm starting to pay attention to memory usage after the experience of
turning off the JIT.
2022-06-10 07:14:45 -07:00
Kartik K. Agaram f554ce878b include a unit test 2022-06-09 18:36:16 -07:00
Kartik K. Agaram 4476eb8946 bugfix: cut (C-x) without first selecting anything 2022-06-09 18:27:09 -07:00
Kartik K. Agaram 6ba10b4de6 fix a corner case when selecting text
The hard part here is keeping click-drag selection working (without
pressing and holding shift).
2022-06-09 15:49:16 -07:00
Kartik K. Agaram fa5bf1218a move 2022-06-09 15:48:07 -07:00
Kartik K. Agaram 8f823bf430 . 2022-06-09 15:22:21 -07:00
Kartik K. Agaram 173d3a3de0 test both ways of selecting text with mouse 2022-06-09 15:09:32 -07:00
Kartik K. Agaram fe170fc579 experiment: blinking cursor
I've tried to keep the time period of the blinking similar to my
terminal.

Honestly I'm no longer sure if any of my experiments are showing a
statistically significant result. Let's see how it feels over a period
of time.
2022-06-09 13:47:41 -07:00
Kartik K. Agaram 25190676ea the problem is that the cursor can occlude text
And anything we do to reduce the occlusion also makes the cursor harder
to acquire.

I suppose this is why we need the blink.
2022-06-09 13:39:41 -07:00
Kartik K. Agaram 06fae39088 make the cursor a little thicker
I'm testing this by moving the cursor around with my eyes closed, then
starting a stopwatch as I open my eyes. This seems to help a bit. I'm
able to acquire the cursor in 2s. At least the 10s outliers I used to
have with the circle or thin line don't seem to be happening.
2022-06-09 13:26:49 -07:00
Kartik K. Agaram b418e60e6a experiment: line cursor 2022-06-09 13:20:05 -07:00
Kartik K. Agaram 7be2718f0f fix contact link 2022-06-09 12:16:03 -07:00
Kartik K. Agaram 712dc95985 experiment: turn off JIT
This seems to speed up copy! What does it slow down?
2022-06-09 08:15:32 -07:00
Kartik K. Agaram 4ad51d648b move 2022-06-09 07:29:38 -07:00
Kartik K. Agaram e77157d316 speeding up copy, attempt 1
Problem: repeatedly copying (relatively large) sections of text quickly
makes the app sluggish until it has to be killed. (Thanks John Blommers
for the report.)

When I instrument with prints, the sluggishness seems to happen in
random draw() calls many times after I perform the copy.

I don't know for sure, but I'm initially checking if the cause is
garbage generated by repeated string concatenation.

This attempt doesn't seem to make any difference.
2022-06-09 07:24:40 -07:00
Kartik K. Agaram ac4879bb85 more defensive resize handling
Thanks John Blommers for the report!
2022-06-07 21:55:48 -07:00
Kartik K. Agaram 12f5fa9bed forgot to commit documentation for the commandline arg 2022-06-07 14:20:54 -07:00
Kartik K. Agaram ff08bbe7f4 first commandline arg: window dimensions
Hopefully there won't be too many others.
2022-06-07 14:11:09 -07:00
Kartik K. Agaram 6b628781d1 use app name in window title 2022-06-07 13:59:26 -07:00
Kartik K. Agaram 7b78c359b6 clearer discription of how to run lines.love 2022-06-07 13:58:38 -07:00