Giter VIP home page Giter VIP logo

Comments (12)

MeSHA256 avatar MeSHA256 commented on June 7, 2024 1

Delphi 11.3. No problems after more than 3 hours of coding.

When LSP took the entire CPU for itself, I got it:
photo_2023-09-27_17-11-05

But I think the problem here is only in the LSP itself, because I have to kill it regularly, otherwise it is impossible to work.
Tomorrow I will test your fix on 10.4.

from delphi-wakatime.

diegomgarcia avatar diegomgarcia commented on June 7, 2024 1

Thats a really great news Michael!

Thanks for testing it and for the feedback.

Lets monitor a few days to see how it will behave.

I will just take a look on how the notifier thread is releasing the wakatime-cli.exe as it seems that it has lots of then alocated on your screenshot, maybe I can improve that also.

Delphi 11.3. No problems after more than 3 hours of coding.

When LSP took the entire CPU for itself, I got it: photo_2023-09-27_17-11-05

But I think the problem here is only in the LSP itself, because I have to kill it regularly, otherwise it is impossible to work. Tomorrow I will test your fix on 10.4.

from delphi-wakatime.

MeSHA256 avatar MeSHA256 commented on June 7, 2024 1

Delphi 10.4. Everything all right. No more problems with CPU overload.

from delphi-wakatime.

diegomgarcia avatar diegomgarcia commented on June 7, 2024 1

Awesome news!!!

Thanks Michael!

from delphi-wakatime.

MeSHA256 avatar MeSHA256 commented on June 7, 2024 1

Awesome news!!!

Thanks Michael!

Thanks for your implementation of wakatime for Delphi :)

from delphi-wakatime.

diegomgarcia avatar diegomgarcia commented on June 7, 2024 1

You are welcome Michael =)

I was needing this to track my projects to report to the customers as we usually do here with C# and other languages. So this was kind of a must have to do job LOL.

Hope everyone like it =)

from delphi-wakatime.

diegomgarcia avatar diegomgarcia commented on June 7, 2024

Thanks for the report @mobius1qwe, I think this is the main cause for the other issue that is crashing the IDE on exit.

I will investigate that with the full logs of FastMM to see if I can track the main issue and solve it. However I'm relocating to another country so I expect to work on this issue and the other ones at the beginning of the next month.

from delphi-wakatime.

mobius1qwe avatar mobius1qwe commented on June 7, 2024

Upon further usage and investigation, the leak (or slowing) seems to happen after a debugging period, if we use the IDE without running in debug mode the code, it doesn't seem to happen.

from delphi-wakatime.

MeSHA256 avatar MeSHA256 commented on June 7, 2024

Upon further usage and investigation, the leak (or slowing) seems to happen after a debugging period, if we use the IDE without running in debug mode the code, it doesn't seem to happen.

I have this problem with CPU about once every 1-2 hours, even without using debug mode. But without memory leaks in Delphi 10.4. Both in Delphi 11.3.

from delphi-wakatime.

diegomgarcia avatar diegomgarcia commented on June 7, 2024

Hey guys,

I've just commited the fix for this issue on the development branch, can you guys test it before I merge with the main?

from delphi-wakatime.

MeSHA256 avatar MeSHA256 commented on June 7, 2024

Hey guys,

I've just commited the fix for this issue on the development branch, can you guys test it before I merge with the main?

Yeah, no problem.

from delphi-wakatime.

diegomgarcia avatar diegomgarcia commented on June 7, 2024

I've just reviewed the NotifierThread code and it doesn't has any pottential leak on it, however I've fixed a typo that wasn't applying one of the cleanup of the command line strings. In some cases depending on some factors, maybe it could fail to report it to wakatime. Also improved some variable naming ...

So for now I think this version is good to go. I've been using since yesterday without problems, will just wait a couple of days to see if we got any other reports of use here so we can validate it.

from delphi-wakatime.

Related Issues (9)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.