Skip to content

Scrolling performance regression using intermediate NSImage #796

Closed

Description

@comex

I'm filing this issue to track the known scrolling performance regression from #757 (see that thread for more details). Although it's known already, this way I can subscribe to the issue and hopefully get notified when it's fixed. :)

Activity

  1. added 2 commits that reference this issue on Dec 5, 2018
  2. added
    RendererText rendering issues, including CoreText renderer
    on Dec 12, 2018
  3. added this to the snapshot-155 milestone on Dec 20, 2018
  4. s4y commented on Feb 16, 2019

    @s4y
    Contributor

    Here's a first pass at a change which significantly improves performance for me. I'm not quite ready to make a PR, but please try it out:

    https://github.com/s4y/macvim/tree/stateful-render

    All feedback is welcome.

  5. s4y commented on Feb 16, 2019

    @s4y
    Contributor

    Two notes re. the above:

    1. Some of the new rendering code is just copied and pasted from MMCoreTextView's three other rendering paths. Since it seems to do as well as them, I'd love to delete the other paths in my PR instead of trying to make them all work at the same time.

    2. I'm not done making rendering faster 😉. If this works out, I know what I want to do next.

  6. amadeus commented on Feb 16, 2019

    @amadeus

    Omg, this is night and day faster!

    I have a very fast keyboard repeat rate and so scroll performance is super noticeable when it chugs!

    Great work so far, excited for what's to come!

    (I'm on vacation right now - so won't have a ton of opportunity to test)

  7. juliuszint commented on Feb 18, 2019

    @juliuszint

    Performance looks great but UltiSnips stopped working with this patch. Otherwise great work!

  8. ychin commented on Feb 18, 2019

    @ychin
    Member

    @juliuszint How does UltiSnips not work? Please file another issue for that. Also there's a new version coming out that fixes Ctrl-C not working but it's hard to tell if it's related without more info.

  9. juliuszint commented on Feb 18, 2019

    @juliuszint

    @ychin Sorry, that was just meant as a feedback for the patch that @s4y provided. Since he explicitly wrote that this code is not ready for a pull request and not merged into master i think filing a issue would be a little bit early. I never encountered this problem with a official MacVim release.

  10. ychin commented on Feb 18, 2019

    @ychin
    Member

    @juliuszint Ah ok. Cool good to know. One less bug for me to fix.

    @s4y if you feel like it I would recommend making a pull request with "[WIP]" as a prefix to the title so conversations and discussions on your fix can be focused there. It's up to you though.

  11. amadeus commented on Feb 19, 2019

    @amadeus

    One small thing I have noticed:

    example

    Every once in a while, a tiny artifact will appear, in roughly the same-ish location. It looks like a dead pixel but I can confirm its not since (a it comes through on screenshots) and be it follows the window when I move it around. If I scroll the document it goes away

  12. 27 remaining items

  13. added a commit that references this issue on Apr 10, 2020
    3413058
  14. modified the milestones: snapshot-163, Backlog on Apr 11, 2020
  15. added a commit that references this issue on May 6, 2020
    47ca47c
  16. gorkunov commented on May 23, 2020

    @gorkunov

    Could someone share build here for test. I cant build it locally due to Error 65 which should be resolved by installing Xcode - but it doesn't help me.

  17. juliuszint commented on May 23, 2020

    @juliuszint

    MacVim.app.zip
    Configured with ./configure --enable-python3interp. Hope this works for you.

  18. eirnym commented on May 24, 2020

    @eirnym
    Contributor

    MacVim.app.zip
    Configured with ./configure --enable-python3interp. Hope this works for you.

    Better to make it with python3 dynamic

  19. added a commit that references this issue on Jul 9, 2020
    93f45be
  20. added a commit that references this issue on Sep 20, 2020
    016b2a4
  21. added 2 commits that reference this issue on Oct 16, 2020
    0a15b50
    3aee6b2
  22. added a commit that references this issue on Dec 3, 2020
    dba6293
  23. modified the milestones: Backlog, snapshot-167 on Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    10.14 MojaveRendererText rendering issues, including CoreText renderer

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions