Forum > General

IDE freezes on SelectDirectory

<< < (3/5) > >>

totya:

--- Quote from: Martin_fr on February 14, 2023, 05:29:09 pm ---The fix was to complex to be merged to fixes. To risky to break something else. So it is 2.3 only.

--- End quote ---

Thanks for the info!

First impression, ide options/key mappings/find key combination filter does not work. This is the first step after lazarus installed...

For example grab ctrl-alt-t, find key combination changed to: "Filter: ctr-alt-t" but the key list not filtered. Lazarus 2.3.0, fpc 3.2.3 (fixes branch).

Martin_fr:

--- Quote from: totya on February 14, 2023, 08:02:14 pm ---
--- Quote from: Martin_fr on February 14, 2023, 05:29:09 pm ---The fix was to complex to be merged to fixes. To risky to break something else. So it is 2.3 only.

--- End quote ---

Thanks for the info!

First impression, ide options/key mappings/find key combination filter does not work. This is the first step after lazarus installed...

For example grab ctrl-alt-t, find key combination changed to: "Filter: ctr-alt-t" but the key list not filtered. Lazarus 2.3.0, fpc 3.2.3 (fixes branch).

--- End quote ---

Indeed, I can confirm this.

I created an issue for it https://gitlab.com/freepascal.org/lazarus/lazarus/-/issues/40121

totya:

--- Quote from: Martin_fr on February 14, 2023, 08:04:09 pm ---Indeed, I can confirm this.
I created an issue for it https://gitlab.com/freepascal.org/lazarus/lazarus/-/issues/40121
--- End quote ---

Thank you!

Martin_fr:
I experimented with merging it.
It's still a big change, and hard to test if nothing was overlooked.

So it's not in the official fixes branch. But I uploaded a copy of the fixes branch at
https://gitlab.com/martin_frb/lazarus/-/commits/test-fpdebug-step-fix-merge

You can get it as follows (from within any clone of either the official Lazarus git, or any fork thereof)

--- Code: Text  [+][-]window.onload = function(){var x1 = document.getElementById("main_content_section"); if (x1) { var x = document.getElementsByClassName("geshi");for (var i = 0; i < x.length; i++) { x[i].style.maxHeight='none'; x[i].style.height = Math.min(x[i].clientHeight+15,306)+'px'; x[i].style.resize = "vertical";}};} ---git fetch https://gitlab.com/martin_frb/lazarus.git test-fpdebug-step-fix-mergegit switch -c mod-fixes-2_2 FETCH_HEAD
If you do, let me know, and let me know if it works for you. Thanks

EDIT:

I have to check, it seems I took an old version of the fix.... May need to change this.

EDIT 2:
Same result, only the ref "cherry picked from ..." in the commit msg is wrong.

---
So you can test it, as described.

totya:

--- Quote from: Martin_fr on February 17, 2023, 10:26:33 am ---I experimented with merging it.

--- End quote ---

Thank you! Its Windows here (no git installed), but torties svn installed, and I use fpcupdeluxe. Can I try your changes with these installed apps anyhow?

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version