Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / lazarus / 2017-05

Lazarus Release Candidate 1 of 1.8.0

61 messages · 11 people · –

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Mattias Gaertner

The Lazarus team is glad to announce the first release candidate of Lazarus 1.8.

This release was built with FPC 3.0.2.
The previous release Lazarus 1.6.4 was built with FPC 3.0.2 too. The Linux releases are built with FPC 3.0.3 r36152 due to bug https://bugs.freepascal.org/view.php?id=31629

Here is the list of fixes for Lazarus 1.8.x:
http://wiki.freepascal.org/Lazarus_1.8_fixes_branch

Here is the list of changes for Lazarus and Free Pascal: http://wiki.lazarus.freepascal.org/Lazarus_1.8.0_release_notes http://wiki.lazarus.freepascal.org/User_Changes_3.0.2

The release is available for download on SourceForge: http://sourceforge.net/projects/lazarus/files/

Choose your CPU, OS, distro and then the "Lazarus 1.8 RC1" directory.

Checksums for the SourceForge files:
http://www.lazarus-ide.org/index.php?page=checksums#1_8_0

Minimum requirements:

Windows:
2k, XP, Vista, 7, 8, 8.1 and 10, 32 or 64bit.

FreeBSD/Linux:
gtk 2.8 for gtk2, qt4.5 for qt, qt5.2 for qt5, 32 or 64bit.

Mac OS X:
10.5 to 10.12, LCL only 32bit, non LCL apps can be 64bit.

The svn tag is
http://svn.freepascal.org/svn/lazarus/tags/lazarus_1_8_0_RC1

For people who are blocked by SF, the Lazarus releases from SourceForge are mirrored at: ftp://freepascal.dfmk.hu/pub/lazarus/releases/
and later at (after some time for synchronization) http://mirrors.iwi.me/lazarus/

== Why should everybody (including you) test the release candidate? ==

In the past weeks the Lazarus team has stabilized the 1.8 fixes branch. The resulting 1.8RC1 is now stable enough to be used by any one for test purposes.

However many of the fixes and new features that where committed since the release of 1.6 required changes to the code of existing features too. While we have tested those ourself, there may still be problems that only occur with very specific configurations or one project in a million.

Yes, it may be that you are the only person with a project, that will not work in the new IDE. So if you do not test, we can not fix it.

Please do not wait for the final release, in order to test. It may be too late. Once the release is out we will have to be more selective about which fixes can be merged for further 1.8.x releases. So it may be, that we can not merge the fix you require. And then you will miss out on all the new features.

== How to test ==

Download and install the 1.8 RC1.
- On Windows you can install as a 2ndary install, that will not affect your current install: http://wiki.lazarus.freepascal.org/Multiple_Lazarus#Installation_of_multiple_Lazarus
- On other platforms, if you install to a new location you need to use --primary-config-path

In either case you should make backups. (including your primary config)

Open your project in the current Lazarus (1.6.x), and use "Publish Project" from the project menu. This creates a clean copy of your project.

You can then open that copy in the RC1. Please test:
- If you can edit forms in the designer
- rename components / change properties in Object inspector / Add new events
- Add components to form / Move components on form
- Frames, if you use them
- If you can navigate the source code (e.g. jump to implementation)
- Auto completion in source code
- Compile, debug and run
- Anything else you use in your daily work

Mattias

Reply by email

zeljko
to Mattias Gaertner
Mattias Gaertner wrote 31 lines
The Lazarus team is glad to announce the first release candidate of Lazarus 1.8. This release was built with FPC 3.0.2. The previous release Lazarus 1.6.4 was built with FPC 3.0.2 too. The Linux releases are built with FPC 3.0.3 r36152 due to bug https://bugs.freepascal.org/view.php?id=31629 Here is the list of fixes for Lazarus 1.8.x: http://wiki.freepascal.org/Lazarus_1.8_fixes_branch Here is the list of changes for Lazarus and Free Pascal: http://wiki.lazarus.freepascal.org/Lazarus_1.8.0_release_notes http://wiki.lazarus.freepascal.org/User_Changes_3.0.2 The release is available for download on SourceForge: http://sourceforge.net/projects/lazarus/files/ Choose your CPU, OS, distro and then the "Lazarus 1.8 RC1" directory. Checksums for the SourceForge files: http://www.lazarus-ide.org/index.php?page=checksums#1_8_0 Minimum requirements: Windows: 2k, XP, Vista, 7, 8, 8.1 and 10, 32 or 64bit. FreeBSD/Linux: gtk 2.8 for gtk2, qt4.5 for qt, qt5.2 for qt5, 32 or 64bit.

Not good. Qt-5.6 is minimum required version (5.6.xx is LTS) not qt5.2 :)

zeljko

Reply by email

Mattias Gaertner

Here is the fixed version with qt5.6 and no dfmk.hu:

The Lazarus team is glad to announce the first release candidate of Lazarus 1.8.

This release was built with FPC 3.0.2.
The previous release Lazarus 1.6.4 was built with FPC 3.0.2 too. The Linux releases are built with FPC 3.0.3 r36152 due to bug https://bugs.freepascal.org/view.php?id=31629

Here is the list of fixes for Lazarus 1.8.x:
http://wiki.freepascal.org/Lazarus_1.8_fixes_branch

Here is the list of changes for Lazarus and Free Pascal: http://wiki.lazarus.freepascal.org/Lazarus_1.8.0_release_notes http://wiki.lazarus.freepascal.org/User_Changes_3.0.2

The release is available for download on SourceForge: http://sourceforge.net/projects/lazarus/files/

Choose your CPU, OS, distro and then the "Lazarus 1.8 RC1" directory.

Checksums for the SourceForge files:
http://www.lazarus-ide.org/index.php?page=checksums#1_8_0

Minimum requirements:

Windows:
2k, XP, Vista, 7, 8, 8.1 and 10, 32 or 64bit.

FreeBSD/Linux:
gtk 2.8 for gtk2, qt4.5 for qt, qt5.6 for qt5, 32 or 64bit.

Mac OS X:
10.5 to 10.12, LCL only 32bit, non LCL apps can be 64bit.

The svn tag is
http://svn.freepascal.org/svn/lazarus/tags/lazarus_1_8_0_RC1

For people who are blocked by SF, the Lazarus releases from SourceForge are mirrored at: ftp://ftp.freepascal.org/pub/lazarus/releases/
and later at (after some time for synchronization) http://mirrors.iwi.me/lazarus/

== Why should everybody (including you) test the release candidate? ==

In the past weeks the Lazarus team has stabilized the 1.8 fixes branch. The resulting 1.8RC1 is now stable enough to be used by any one for test purposes.

However many of the fixes and new features that where committed since the release of 1.6 required changes to the code of existing features too. While we have tested those ourself, there may still be problems that only occur with very specific configurations or one project in a million.

Yes, it may be that you are the only person with a project, that will not work in the new IDE. So if you do not test, we can not fix it.

Please do not wait for the final release, in order to test. It may be too late. Once the release is out we will have to be more selective about which fixes can be merged for further 1.8.x releases. So it may be, that we can not merge the fix you require. And then you will miss out on all the new features.

== How to test ==

Download and install the 1.8 RC1.
- On Windows you can install as a 2ndary install, that will not affect your current install: http://wiki.lazarus.freepascal.org/Multiple_Lazarus#Installation_of_multiple_Lazarus
- On other platforms, if you install to a new location you need to use --primary-config-path

In either case you should make backups. (including your primary config)

Open your project in the current Lazarus (1.6.x), and use "Publish Project" from the project menu. This creates a clean copy of your project.

You can then open that copy in the RC1. Please test:
- If you can edit forms in the designer
- rename components / change properties in Object inspector / Add new events
- Add components to form / Move components on form
- Frames, if you use them
- If you can navigate the source code (e.g. jump to implementation)
- Auto completion in source code
- Compile, debug and run
- Anything else you use in your daily work

Mattias

Reply by email

Luca Olivetti
to Mattias Gaertner: The Lazarus team is glad to announce the first release candidate of Lazarus 1.8. This…

El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit:

== Why should everybody (including you) test the release candidate? ==

I didn't test it yet but I checked it out from svn and I see that the patch here

https://bugs.freepascal.org/view.php?id=30931

isn't included.

Is this version still supposed to work with fpc 2.6.4?

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Luca Olivetti
to Luca Olivetti

El 17/05/17 a les 16:31, Luca Olivetti via Lazarus ha escrit:

Show quoted text 12 lines
El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit: > == Why should everybody (including you) test the release candidate? == I didn't test it yet but I checked it out from svn and I see that the patch here https://bugs.freepascal.org/view.php?id=30931 isn't included. Is this version still supposed to work with fpc 2.6.4?

OK, I see now that it doesn't build with fpc 2.6.4 :-(

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Mattias Gaertner
to Luca Olivetti: El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit: I didn't test it yet but…
Show quoted text 12 lines
El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit: > == Why should everybody (including you) test the release candidate? == I didn't test it yet but I checked it out from svn and I see that the patch here https://bugs.freepascal.org/view.php?id=30931 isn't included. Is this version still supposed to work with fpc 2.6.4?

Yes.

I applied the patch. Thank You!

Mattias

Reply by email

Luca Olivetti
to Mattias Gaertner

El 17/05/17 a les 16:39, Mattias Gaertner via Lazarus ha escrit:

Show quoted text 17 lines
On Wed, 17 May 2017 16:31:33 +0200 Luca Olivetti via Lazarus wrote: > El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit: > >> == Why should everybody (including you) test the release candidate? == > > I didn't test it yet but I checked it out from svn and I see that the > patch here > > https://bugs.freepascal.org/view.php?id=30931 > > isn't included. > > Is this version still supposed to work with fpc 2.6.4? Yes.

Well, in that case you'll have to comment/ifdef the use of "Default" in components/codetools/finddeclarationtool.pas
It seems it's just used to silence a warning.

I applied the patch. Thank You!

Thank you

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Luca Olivetti
to Mattias Gaertner

El 17/05/17 a les 16:48, Mattias Gaertner via Lazarus ha escrit:

Show quoted text 13 lines
On Wed, 17 May 2017 16:34:14 +0200 Luca Olivetti via Lazarus wrote: > [...] >> Is this version still supposed to work with fpc 2.6.4? > > OK, I see now that it doesn't build with fpc 2.6.4 :-( Fixed. The Lazarus devels only support the last two FPC releases, which means 3.0.0 and 3.0.2. But as long as FPC 2.6.4 only needs a few IFDEFs, patches are welcome.

Ok.
A thing that broke from lazarus 1.6.4 is TLabel.OptimalFill. It seems that, depending on the width of the label it just uses a tiny amount of space, e.g., I put a label with the caption xxxx, height 140. With width 200 the four x fill all the horizontal space, if I change it to 300 they just occupy the top left corner (see attached image). See also the bogus handles (is that the correct name?) in the middle of the label. I saw those for quite a while (in lazarus 1.6.4) but it didn't bother me.
This is on windows 7 32 bits.

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007 -------------- next part -------------- A non-text attachment was scrubbed... Name: optimalfill.png Type: image/png Size: 4479 bytes Desc: not available URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170517/3aa83045/attachment.png>

Reply by email

Luca Olivetti
to Luca Olivetti

El 17/05/17 a les 17:29, Luca Olivetti via Lazarus ha escrit:

Show quoted text 21 lines
El 17/05/17 a les 16:48, Mattias Gaertner via Lazarus ha escrit: > On Wed, 17 May 2017 16:34:14 +0200 > Luca Olivetti via Lazarus wrote: > >> [...] >>> Is this version still supposed to work with fpc 2.6.4? >> >> OK, I see now that it doesn't build with fpc 2.6.4 :-( > > Fixed. > > The Lazarus devels only support the last two FPC releases, which means > 3.0.0 and 3.0.2. > But as long as FPC 2.6.4 only needs a few IFDEFs, patches are welcome. Ok. A thing that broke from lazarus 1.6.4 is TLabel.OptimalFill. It seems that, depending on the width of the label it just uses a tiny amount of space, e.g., I put a label with the caption xxxx, height 140. With width 200 the four x fill all the horizontal space, if I change it to 300 they just occupy the top left corner (see attached image).

FWIW it does the same with linux/gtk2 and, boy, the ide is *ugly*: giant, blurred, icons everywhere, giant fonts in the tabs of the components' palette, giant fonts in the object inspector...you get the drift.
The fonts in the editor seems fine, but maybe it's because I changed the default a long time ago.

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007 -------------- next part -------------- A non-text attachment was scrubbed... Name: uglyide.png Type: image/png Size: 61670 bytes Desc: not available URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170517/3e92986b/attachment-0001.png>

Reply by email

Ondrej Pokorny
to Luca Olivetti
Luca Olivetti wrote 6 lines
FWIW it does the same with linux/gtk2 and, boy, the ide is *ugly*: giant, blurred, icons everywhere, giant fonts in the tabs of the components' palette, giant fonts in the object inspector...you get the drift. The fonts in the editor seems fine, but maybe it's because I changed the default a long time ago.

What is your DPI settings?

Ondrej

Reply by email

Luca Olivetti
to Ondrej Pokorny

El 17/05/17 a les 17:43, Ondrej Pokorny via Lazarus ha escrit:

Show quoted text 9 lines
On 17.05.2017 17:38, Luca Olivetti via Lazarus wrote: > FWIW it does the same with linux/gtk2 and, boy, the ide is *ugly*: > giant, blurred, icons everywhere, giant fonts in the tabs of the > components' palette, giant fonts in the object inspector...you get the > drift. > The fonts in the editor seems fine, but maybe it's because I changed > the default a long time ago. What is your DPI settings?

Is this the correct way to check?
$ xdpyinfo | grep dots
resolution: 143x144 dots per inch

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Ondrej Pokorny
to Luca Olivetti
Luca Olivetti wrote 12 lines
El 17/05/17 a les 17:43, Ondrej Pokorny via Lazarus ha escrit: > On 17.05.2017 17:38, Luca Olivetti via Lazarus wrote: >> FWIW it does the same with linux/gtk2 and, boy, the ide is *ugly*: >> giant, blurred, icons everywhere, giant fonts in the tabs of the >> components' palette, giant fonts in the object inspector...you get >> the drift. >> The fonts in the editor seems fine, but maybe it's because I changed >> the default a long time ago. > > What is your DPI settings? Is this the correct way to check?

Yes it is.

$ xdpyinfo | grep dots resolution: 143x144 dots per inch

You use 150% scaling (96 DPI = 100%), so Lazarus is also scaled by 150%.

It's a new feature :)

Ondrej

Reply by email

Luca Olivetti
to Ondrej Pokorny

El 17/05/17 a les 17:52, Ondrej Pokorny via Lazarus ha escrit:

Show quoted text 22 lines
On 17.05.2017 17:47, Luca Olivetti via Lazarus wrote: > El 17/05/17 a les 17:43, Ondrej Pokorny via Lazarus ha escrit: >> On 17.05.2017 17:38, Luca Olivetti via Lazarus wrote: >>> FWIW it does the same with linux/gtk2 and, boy, the ide is *ugly*: >>> giant, blurred, icons everywhere, giant fonts in the tabs of the >>> components' palette, giant fonts in the object inspector...you get >>> the drift. >>> The fonts in the editor seems fine, but maybe it's because I changed >>> the default a long time ago. >> >> What is your DPI settings? > > Is this the correct way to check? Yes it is. > $ xdpyinfo | grep dots > resolution: 143x144 dots per inch You use 150% scaling (96 DPI = 100%), so Lazarus is also scaled by 150%. It's a new feature :)

Well, if it is, it's broken: every other application (either kde or gtk) shows fine, the only ugly one is lazarus :-(
OTOH in the plasma control panel I see no option to set the dpi, the only one I could find is "screen scaling" which is set at the lowest possible value (1), if I set it to 3 the preview shows elements (fonts, tabs, buttons, etc.) 3 times as big (duh!).
There's a section to select the gtk theme but it doesn't have an option to select the dpi either (just the gtk2 theme, the gtk3 one and the font).

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Ondrej Pokorny
to Luca Olivetti
Luca Olivetti wrote 14 lines
El 17/05/17 a les 17:52, Ondrej Pokorny via Lazarus ha escrit: > You use 150% scaling (96 DPI = 100%), so Lazarus is also scaled by 150%. > > It's a new feature :) Well, if it is, it's broken: every other application (either kde or gtk) shows fine, the only ugly one is lazarus :-( OTOH in the plasma control panel I see no option to set the dpi, the only one I could find is "screen scaling" which is set at the lowest possible value (1), if I set it to 3 the preview shows elements (fonts, tabs, buttons, etc.) 3 times as big (duh!). There's a section to select the gtk theme but it doesn't have an option to select the dpi either (just the gtk2 theme, the gtk3 one and the font).

It looks to me like this issue:
https://mantis.freepascal.org/view.php?id=31199

Linux is doing something very not-standard again :/ (Having various settings for DPI...)

Ondrej

Reply by email

Luca Olivetti
to Luca Olivetti: El 17/05/17 a les 17:52, Ondrej Pokorny via Lazarus ha escrit: Well, if it is, it's…

El 17/05/17 a les 18:55, Luca Olivetti via Lazarus ha escrit:

Show quoted text 32 lines
El 17/05/17 a les 17:52, Ondrej Pokorny via Lazarus ha escrit: > On 17.05.2017 17:47, Luca Olivetti via Lazarus wrote: >> El 17/05/17 a les 17:43, Ondrej Pokorny via Lazarus ha escrit: >>> On 17.05.2017 17:38, Luca Olivetti via Lazarus wrote: >>>> FWIW it does the same with linux/gtk2 and, boy, the ide is *ugly*: >>>> giant, blurred, icons everywhere, giant fonts in the tabs of the >>>> components' palette, giant fonts in the object inspector...you get >>>> the drift. >>>> The fonts in the editor seems fine, but maybe it's because I >>>> changed the default a long time ago. >>> >>> What is your DPI settings? >> >> Is this the correct way to check? > > Yes it is. > >> $ xdpyinfo | grep dots >> resolution: 143x144 dots per inch > > You use 150% scaling (96 DPI = 100%), so Lazarus is also scaled by 150%. > > It's a new feature :) Well, if it is, it's broken: every other application (either kde or gtk) shows fine, the only ugly one is lazarus :-( OTOH in the plasma control panel I see no option to set the dpi, the only one I could find is "screen scaling" which is set at the lowest possible value (1), if I set it to 3 the preview shows elements (fonts, tabs, buttons, etc.) 3 times as big (duh!). There's a section to select the gtk theme but it doesn't have an option to select the dpi either (just the gtk2 theme, the gtk3 one and the font).

The ide compiled with the qt widgetset doesn't show the same problem (though it doesn't seem to be using the same fonts as other kde applications)

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007 -------------- next part -------------- A non-text attachment was scrubbed... Name: notsouglybutnotquiteright.png Type: image/png Size: 20700 bytes Desc: not available URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170517/b4924df9/attachment-0001.png>

Reply by email

zeljko
to Luca Olivetti
Luca Olivetti wrote
The ide compiled with the qt widgetset doesn't show the same problem (though it doesn't seem to be using the same fonts as other kde applications)

KDE Plasma is using Qt5, but ide is Qt4, so in Qt4 config you can set same fonts as they are in KDE Plasma and they will be exactly same. It's bug in KDE, since applying any setting to Qt5 (via KDE settings), should apply same applicable settings for Qt4 applications. Besides that, you can build cbindings for Qt5 and rebuild lazarus against Qt5 widgetset to see if there's any difference.

zeljko

Reply by email

Luca Olivetti
to Ondrej Pokorny: It looks to me like this issue: https://mantis.freepascal.org/view.php?id=31199 Linux is…

El 17/05/17 a les 19:05, Ondrej Pokorny via Lazarus ha escrit:

Show quoted text 21 lines
On 17.05.2017 18:55, Luca Olivetti via Lazarus wrote: > El 17/05/17 a les 17:52, Ondrej Pokorny via Lazarus ha escrit: >> You use 150% scaling (96 DPI = 100%), so Lazarus is also scaled by 150%. >> >> It's a new feature :) > > Well, if it is, it's broken: every other application (either kde or > gtk) shows fine, the only ugly one is lazarus :-( > OTOH in the plasma control panel I see no option to set the dpi, the > only one I could find is "screen scaling" which is set at the lowest > possible value (1), if I set it to 3 the preview shows elements > (fonts, tabs, buttons, etc.) 3 times as big (duh!). > There's a section to select the gtk theme but it doesn't have an > option to select the dpi either (just the gtk2 theme, the gtk3 one and > the font). It looks to me like this issue: https://mantis.freepascal.org/view.php?id=31199 Linux is doing something very not-standard again :/ (Having various settings for DPI...)

Could be (though I'm using kubuntu instead of ubuntu so the settings are different) but every other application shows (apparently) fine....

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Luca Olivetti
to zeljko: KDE Plasma is using Qt5, but ide is Qt4, so in Qt4 config you can set same fonts as they…

El 17/05/17 a les 19:19, zeljko ha escrit:

Show quoted text 12 lines
On 17.05.2017 19:08, Luca Olivetti via Lazarus wrote: > The ide compiled with the qt widgetset doesn't show the same problem > (though it doesn't seem to be using the same fonts as other kde > applications) KDE Plasma is using Qt5, but ide is Qt4, so in Qt4 config you can set same fonts as they are in KDE Plasma and they will be exactly same. It's bug in KDE, since applying any setting to Qt5 (via KDE settings), should apply same applicable settings for Qt4 applications. Besides that, you can build cbindings for Qt5 and rebuild lazarus against Qt5 widgetset to see if there's any difference.

It took me a while to see that the c bindings are included, the README.txt in the lcl/interfaces/qt5 directory just says:

"To use them you also need to download and install at appropriate place the libQt5Pas.so (linux)
or Qt5Pas.dll (ms windows) or Qt5Pas.framework (Darwin).

For more information see:
http://wiki.lazarus.freepascal.org/index.php/Qt_Interface"

So, after unsuccessfully compiling the beta version from http://users.telenet.be/Jan.Van.hijfte/qtforfpc/fpcqt4.html I finally found http://wiki.lazarus.freepascal.org/Qt5_Interface which says that the bindings are included :-/
They're building now.

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Luca Olivetti
to Luca Olivetti

El 17/05/17 a les 19:59, Luca Olivetti via Lazarus ha escrit:

Show quoted text 30 lines
El 17/05/17 a les 19:19, zeljko ha escrit: > On 17.05.2017 19:08, Luca Olivetti via Lazarus wrote: > >> The ide compiled with the qt widgetset doesn't show the same problem >> (though it doesn't seem to be using the same fonts as other kde >> applications) > > KDE Plasma is using Qt5, but ide is Qt4, so in Qt4 config you can set > same fonts as they are in KDE Plasma and they will be exactly same. > It's bug in KDE, since applying any setting to Qt5 (via KDE settings), > should apply same applicable settings for Qt4 applications. > Besides that, you can build cbindings for Qt5 and rebuild lazarus > against Qt5 widgetset to see if there's any difference. It took me a while to see that the c bindings are included, the README.txt in the lcl/interfaces/qt5 directory just says: "To use them you also need to download and install at appropriate place the libQt5Pas.so (linux) or Qt5Pas.dll (ms windows) or Qt5Pas.framework (Darwin). For more information see: http://wiki.lazarus.freepascal.org/index.php/Qt_Interface" So, after unsuccessfully compiling the beta version from http://users.telenet.be/Jan.Van.hijfte/qtforfpc/fpcqt4.html I finally found http://wiki.lazarus.freepascal.org/Qt5_Interface which says that the bindings are included :-/ They're building now.

Excellent, this is OK, with my selected fonts and correctly sized icons. It still has the problem with optimalfill, so it's definitely not platform specific.

Meanwhile I see that in the plasma font setting I have the dpi forced to 96, a setting that apparently is respected both by qt4 and qt5 (as well as gtk applications).

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

zeljko
to Luca Olivetti: El 17/05/17 a les 19:19, zeljko ha escrit: It took me a while to see that the c bindings…
Luca Olivetti wrote 15 lines
It took me a while to see that the c bindings are included, the README.txt in the lcl/interfaces/qt5 directory just says: "To use them you also need to download and install at appropriate place the libQt5Pas.so (linux) or Qt5Pas.dll (ms windows) or Qt5Pas.framework (Darwin). For more information see: http://wiki.lazarus.freepascal.org/index.php/Qt_Interface" So, after unsuccessfully compiling the beta version from http://users.telenet.be/Jan.Van.hijfte/qtforfpc/fpcqt4.html I finally found http://wiki.lazarus.freepascal.org/Qt5_Interface which says that the bindings are included :-/ They're building now.

Thanks, I'll fix README.txt ... Jan stopped to support bindings for almost one year, so I've taken his latest Qt5 bindings an manually updated classes and everything and commited it into laz trunk.. You must build it at your own for now, but as soon as I stop update C part of bindings I'll build them for linux64,win32 and macosx qtcocoa 64bit and put it somewhere so ppl can download. I don't have any plans to build linux32 or mac32 (carbon) Qt5 C bindings atm.

zeljko

Reply by email

Luca Olivetti
to zeljko

El 17/05/17 a les 20:33, zeljko ha escrit:

Show quoted text 9 lines
On 17.05.2017 20:16, Luca Olivetti via Lazarus wrote: > Meanwhile I see that in the plasma font setting I have the dpi forced to > 96, a setting that apparently is respected both by qt4 and qt5 (as well > as gtk applications). If it's forced than all qt based apps are using forced 96dpi. Even if you set some other dpi in console session, qtlcl app uses forced 96 dpi.

Yes, I unchecked the option and now even lazarus/qt5 is ugly :-) Now I see why I set (and forgot about) the option ;-)

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Joe
to Luca Olivetti: El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit: I didn't test it yet but…
Luca Olivetti wrote 14 lines
El 17/05/17 a les 14:39, Mattias Gaertner via Lazarus ha escrit: > == Why should everybody (including you) test the release candidate? == I didn't test it yet but I checked it out from svn and I see that the patch here https://bugs.freepascal.org/view.php?id=30931 isn't included. Is this version still supposed to work with fpc 2.6.4? Bye

Is this the correct way to report my finding? As a non professional I am not sure.

Lazarus 1.8 RC1 under Win 10 (64):
"GetEnvironmentVariable('computername')" doesn't work. When compiling, it returns "Error: Wrong number of Parameters specified for call to GetEnvironmentVariable".

Reply by email

Andreas Frieß
to Mattias Gaertner: The Lazarus team is glad to announce the first release candidate of Lazarus 1.8. This…

I see with the last patches something goes wrong after r54880 with frames and grid (specially TsWorkSheetGrid).

Lazarus claims, the is no valid parent and didnot load and crash. It is only happen if you use frames ? The trouble is, i can only shutdown Lazarus with the Processmanger.

Win10/64 with Lazarus 32/svn 1.9RC1

Andreas

Any hints ?

Mattias Gaertner wrote
The Lazarus team is glad to announce the first release candidate of Lazarus 1.8.

Reply by email

Kostas Michalopoulos
to Andreas Frieß

I've noticed that the window shrinking bug in window maker with Gtk2 is still there, but i finally managed to figure it out and made a fix. I originally wrote the fix in 1.8RC1 so it should apply there, but the patch is against SVN. I posted a more detailed email in the mailing list, but here is the Mantis link https://bugs.freepascal.org/view.php?id=31832

I also see another issue with the form designer - when i have two forms open, despite which one i activated last, the one i created something first is always being focused even if i try to create something in the other form. Note that this happens in 1.8RC1 and SVN with and without the patch i mentioned above.

Kostas

On Thu, May 18, 2017 at 12:08 AM, Andreas Frieß via Lazarus <

Show quoted text 29 lines
wrote: I see with the last patches something goes wrong after r54880 with frames and grid (specially TsWorkSheetGrid). Lazarus claims, the is no valid parent and didnot load and crash. It is only happen if you use frames ? The trouble is, i can only shutdown Lazarus with the Processmanger. Win10/64 with Lazarus 32/svn 1.9RC1 Andreas Any hints ? Am 17.05.2017 um 14:39 schrieb Mattias Gaertner via Lazarus: > The Lazarus team is glad to announce the first release candidate of > Lazarus 1.8. > > > -- _______________________________________________ Lazarus mailing list http://lists.lazarus-ide.org/listinfo/lazarus

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170518/49444077/attachment.html>

Reply by email

Michael Van Canneyt
to Kostas Michalopoulos
Kostas Michalopoulos wrote 5 lines
I've noticed that the window shrinking bug in window maker with Gtk2 is still there, but i finally managed to figure it out and made a fix. I originally wrote the fix in 1.8RC1 so it should apply there, but the patch is against SVN. I posted a more detailed email in the mailing list, but here is the Mantis link https://bugs.freepascal.org/view.php?id=31832

I think it would be very good to apply this patch for the release.

Michael.

Reply by email

Mattias Gaertner
to Joe: Is this the correct way to report my finding? As a non professional I am not sure.…
Show quoted text 5 lines
[...] Is this the correct way to report my finding? As a non professional I am not sure. Lazarus 1.8 RC1 under Win 10 (64):

Yes, although this is more exact:
Help / About Lazarus / Right mouse click on logo / Copy version info to clipboard.

"GetEnvironmentVariable('computername')" doesn't work. When compiling, it returns "Error: Wrong number of Parameters specified for call to GetEnvironmentVariable".

The SysUtils GetEnvironmentVariable expects one string. So, maybe you have somewhere another GetEnvironmentVariable?
Use Find Declaration to find out.

Mattias

Reply by email

Werner Pamler
to Andreas Frieß: I see with the last patches something goes wrong after r54880 with frames and grid…
Andreas Frieß wrote 8 lines
I see with the last patches something goes wrong after r54880 with frames and grid (specially TsWorkSheetGrid). Lazarus claims, the is no valid parent and didnot load and crash. It is only happen if you use frames ? The trouble is, i can only shutdown Lazarus with the Processmanger. Win10/64 with Lazarus 32/svn 1.9RC1

This is confusing: Are you using Laz trunk (v1.9) or 1.8RC1? I don't see an issue with WorksheetGrid on frames with Laz trunk. Can you post a demo?

Reply by email

Mattias Gaertner
to Andreas Frieß: I see with the last patches something goes wrong after r54880 with frames and grid…
Show quoted text 12 lines
I see with the last patches something goes wrong after r54880 with frames and grid (specially TsWorkSheetGrid). Lazarus claims, the is no valid parent and didnot load and crash. It is only happen if you use frames ? The trouble is, i can only shutdown Lazarus with the Processmanger. Win10/64 with Lazarus 32/svn 1.9RC1 Andreas Any hints ?

Please create a bug report.

Mattias

Reply by email

Joe
to Mattias Gaertner: On Wed, 17 May 2017 22:27:20 +0200 Yes, although this is more exact: Help / About Lazarus…
Mattias Gaertner wrote 21 lines
On Wed, 17 May 2017 22:27:20 +0200 Joe via Lazarus wrote: > [...] > Is this the correct way to report my finding? As a non professional I am > not sure. > > Lazarus 1.8 RC1 under Win 10 (64): Yes, although this is more exact: Help / About Lazarus / Right mouse click on logo / Copy version info to clipboard. > "GetEnvironmentVariable('computername')" doesn't work. > When compiling, it returns "Error: Wrong number of Parameters specified > for call to GetEnvironmentVariable". The SysUtils GetEnvironmentVariable expects one string. So, maybe you have somewhere another GetEnvironmentVariable? Use Find Declaration to find out. Mattias

Mattias,

Find Declaration shows two entries in file osutilsh.inc, lines 25 and 26:
Function GetEnvironmentVariable(Const EnvVar : AnsiString) : AnsiString;
Function GetEnvironmentVariable(Const EnvVar : UnicodeString) :
UnicodeString;

But Code Completion offers three functions named GetEnvironmentVariable: The Ansistring function and the Unicode String function above plus "GetEnvironmentVariable(lpName: LPCSTR; lpBuffer: LPSTR; nSize: DWORD): DWORD".

Seems, Lazarus tries to use only the latter.

No, I don't have another GetEnvironmentVariable.

You can reproduce this easily:
Put a TButton Button1 on a form.
In the OnClick Event for this button write "Button1.Caption := GetEnvironmentVariable ('computername');"

Version Info (Thank you for this hint):
Lazarus 1.8.0RC1 r54919 FPC 3.0.2 x86_64-win64-win32/win64

Joe
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170518/7dce259f/attachment.html>

Reply by email

Martin Frb
to Joe
Joe wrote
But Code Completion offers three functions named GetEnvironmentVariable: The Ansistring function and the Unicode String function above plus "GetEnvironmentVariable(lpName: LPCSTR; lpBuffer: LPSTR; nSize: DWORD): DWORD".

Looks like you may have the unit "windows" in your uses clause.

Reply by email

Ondrej Pokorny
to Andreas Frieß: I see with the last patches something goes wrong after r54880 with frames and grid…
Andreas Frieß wrote 12 lines
I see with the last patches something goes wrong after r54880 with frames and grid (specially TsWorkSheetGrid). Lazarus claims, the is no valid parent and didnot load and crash. It is only happen if you use frames ? The trouble is, i can only shutdown Lazarus with the Processmanger. Win10/64 with Lazarus 32/svn 1.9RC1 Andreas Any hints ?

There may be troubles with the scaling code (that needs a parent).

As Mattias already said - please report to mantis with a test application, thanks!

Ondrej

Reply by email

Andreas
to Ondrej Pokorny

THX, this is a good hint for me ‚scaling code‘, so i can isolate that issue and test. If i can create a testapp, so send it to mantis.

Andreas

Gesendet von Mail für Windows 10

Von: Ondrej Pokorny via Lazarus
Gesendet: Donnerstag, 18. Mai 2017 07:26
An: Lazarus mailing list
Cc: Ondrej Pokorny
Betreff: Re: [Lazarus] Lazarus Release Candidate 1 of 1.8.0

Andreas Frieß wrote 12 lines
I see with the last patches something goes wrong after r54880 with frames and grid (specially TsWorkSheetGrid). Lazarus claims, the is no valid parent and didnot load and crash. It is only happen if you use frames ? The trouble is, i can only shutdown Lazarus with the Processmanger. Win10/64 with Lazarus 32/svn 1.9RC1 Andreas Any hints ?

There may be troubles with the scaling code (that needs a parent).

As Mattias already said - please report to mantis with a test application, thanks!

Ondrej

Reply by email

Luca Olivetti
to Luca Olivetti: El 17/05/17 a les 20:16, Luca Olivetti via Lazarus ha escrit: See…

El 17/05/17 a les 20:22, Luca Olivetti via Lazarus ha escrit:

Show quoted text 8 lines
El 17/05/17 a les 20:16, Luca Olivetti via Lazarus ha escrit: > Excellent, this is OK, with my selected fonts and correctly sized icons. > It still has the problem with optimalfill, so it's definitely not > platform specific. See https://youtu.be/NFZGVx7V2C0

FWIW if I revert the implementation of CalcFittingFontHeight (lcl/include/customlabel.inc) to the one in lazarus 1.6.4 this doesn't happen anymore.
I'll try to see what changed and what broke it.

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Mattias Gaertner
to Martin Frb: Looks like you may have the unit "windows" in your uses clause.
Show quoted text 7 lines
On 18/05/2017 00:34, Joe via Lazarus wrote: > But Code Completion offers three functions named GetEnvironmentVariable: > The Ansistring function and the Unicode String function above plus > "GetEnvironmentVariable(lpName: LPCSTR; lpBuffer: LPSTR; nSize: > DWORD): DWORD". Looks like you may have the unit "windows" in your uses clause.

Either move the Windows unit in front of the SysUtils unit, or use SysUtils.GetEnvironmentVariable

Mattias

Reply by email

Luca Olivetti
to Luca Olivetti: El 17/05/17 a les 20:22, Luca Olivetti via Lazarus ha escrit: FWIW if I revert the…

El 18/05/17 a les 09:17, Luca Olivetti via Lazarus ha escrit:

Show quoted text 13 lines
El 17/05/17 a les 20:22, Luca Olivetti via Lazarus ha escrit: > El 17/05/17 a les 20:16, Luca Olivetti via Lazarus ha escrit: >> Excellent, this is OK, with my selected fonts and correctly sized icons. >> It still has the problem with optimalfill, so it's definitely not >> platform specific. > > > See https://youtu.be/NFZGVx7V2C0 > FWIW if I revert the implementation of CalcFittingFontHeight (lcl/include/customlabel.inc) to the one in lazarus 1.6.4 this doesn't happen anymore. I'll try to see what changed and what broke it.

This seems to fix it, maybe fpc 2.6.4 doesn't manage well a "in" with a range of integers?

Index: lcl/include/customlabel.inc
=================================================================== --- lcl/include/customlabel.inc (revision 54949)
+++ lcl/include/customlabel.inc (working copy)
@@ -355,7 +355,7 @@

          NeededWidth := R.Right - R.Left;
          NeededHeight := R.Bottom - R.Top;
          //debugln('TCustomLabel.CalcFittingFontHeight B

NeededWidth=',dbgs(NeededWidth),' NeededHeight=',dbgs(NeededHeight),' MaxWidth=',dbgs(MaxWidth),' MaxHeight=',dbgs(MaxHeight));
- if (NeededWidth in [1..MaxWidth]) and (NeededHeight in [1..MaxHeight]) then
+ if (NeededWidth > 0) and (NeededWidth <= MaxWidth) and (NeededHeight > 0) and (NeededHeight <= MaxHeight) then

          begin
            // TheText fits into the bounds
            if (not Result) or (FontHeight < TestFont.Height) then
Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Michael Van Canneyt
to Luca Olivetti
Luca Olivetti wrote 17 lines
El 18/05/17 a les 09:17, Luca Olivetti via Lazarus ha escrit: > El 17/05/17 a les 20:22, Luca Olivetti via Lazarus ha escrit: >> El 17/05/17 a les 20:16, Luca Olivetti via Lazarus ha escrit: >>> Excellent, this is OK, with my selected fonts and correctly sized icons. >>> It still has the problem with optimalfill, so it's definitely not >>> platform specific. >> >> >> See https://youtu.be/NFZGVx7V2C0 >> > FWIW if I revert the implementation of CalcFittingFontHeight > (lcl/include/customlabel.inc) to the one in lazarus 1.6.4 this doesn't > happen anymore. > I'll try to see what changed and what broke it. This seems to fix it, maybe fpc 2.6.4 doesn't manage well a "in" with a range of integers?

An "In" was limited to a range of 0..255.

As far as I know, this is still so, but a compiler guru will need to confirm.

If (NeededWidth in [1..MaxWidth]) was used for speed reasons, I'm not sure that that is a good idea.

Michael.

Show quoted text 31 lines
Index: lcl/include/customlabel.inc =================================================================== --- lcl/include/customlabel.inc (revision 54949) +++ lcl/include/customlabel.inc (working copy) @@ -355,7 +355,7 @@ NeededWidth := R.Right - R.Left; NeededHeight := R.Bottom - R.Top; //debugln('TCustomLabel.CalcFittingFontHeight B NeededWidth=',dbgs(NeededWidth),' NeededHeight=',dbgs(NeededHeight),' MaxWidth=',dbgs(MaxWidth),' MaxHeight=',dbgs(MaxHeight)); - if (NeededWidth in [1..MaxWidth]) and (NeededHeight in [1..MaxHeight]) then + if (NeededWidth > 0) and (NeededWidth <= MaxWidth) and (NeededHeight > 0) and (NeededHeight <= MaxHeight) then begin // TheText fits into the bounds if (not Result) or (FontHeight < TestFont.Height) then -- Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007 -- _______________________________________________ Lazarus mailing list http://lists.lazarus-ide.org/listinfo/lazarus

Reply by email

Luca Olivetti
to Ondrej Pokorny

El 18/05/17 a les 10:47, Ondrej Pokorny via Lazarus ha escrit:

Show quoted text 6 lines
On 18.05.2017 9:38, Luca Olivetti via Lazarus wrote: > This seems to fix it, maybe fpc 2.6.4 doesn't manage well a "in" with > a range of integers? Thank you for the patch, applied! Does the bug have a report in mantis so that I can resolve it?

No, I just reported it here.

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Joe
to Mattias Gaertner: On Thu, 18 May 2017 01:24:39 +0100 Either move the Windows unit in front of the SysUtils…
Mattias Gaertner wrote 13 lines
On Thu, 18 May 2017 01:24:39 +0100 Martin Frb via Lazarus wrote: > On 18/05/2017 00:34, Joe via Lazarus wrote: >> But Code Completion offers three functions named GetEnvironmentVariable: >> The Ansistring function and the Unicode String function above plus >> "GetEnvironmentVariable(lpName: LPCSTR; lpBuffer: LPSTR; nSize: >> DWORD): DWORD". > Looks like you may have the unit "windows" in your uses clause. Either move the Windows unit in front of the SysUtils unit, or use SysUtils.GetEnvironmentVariable Mattias

Yes, both solutions work; thank you, Martin and Mattias. Are they documented anywhere?
Joe

Reply by email

Mattias Gaertner
to Joe
Show quoted text 6 lines
[...] > Either move the Windows unit in front of the SysUtils unit, or use > SysUtils.GetEnvironmentVariable Yes, both solutions work; thank you, Martin and Mattias. Are they documented anywhere?

The rule is simple:
If an identifier is searched via the uses section, and it does not start with a unitname, the interfaces of the units of the uses section are searched from right to the left.

Michael, where are the uses section resolve rules explained? I would expect the documentation of the uses section somewhere here:
https://www.freepascal.org/docs-html/current/ref/refse106.html#x216-23800016.2

Btw, the docs about Dotted units
https://www.freepascal.org/docs-html/current/ref/refse107.html states:
"When resolving symbols, unit scopes always take precedence over symbols inside units."

IMO this is misleading. For example:

uses foo;
type
  TMy = class
    foo: TMy;
    procedure DoSome;
  end;
procedure TMy.DoSome;
begin
  foo:=nil; // nearest symbol wins
  foo.foo:=nil; // symbol inside unit wins
end;

The rule is more like this:
If the first symbol of a dotted identifier is an unit in the uses section, the longest match wins.

Mattias

Reply by email

Michael Van Canneyt
to Mattias Gaertner
Mattias Gaertner wrote 17 lines
On Thu, 18 May 2017 13:46:07 +0200 Joe via Lazarus wrote: > [...] > > Either move the Windows unit in front of the SysUtils unit, or use > > SysUtils.GetEnvironmentVariable > > Yes, both solutions work; thank you, Martin and Mattias. Are they > documented anywhere? The rule is simple: If an identifier is searched via the uses section, and it does not start with a unitname, the interfaces of the units of the uses section are searched from right to the left. Michael, where are the uses section resolve rules explained?
I would expect the documentation of the uses section somewhere here: https://www.freepascal.org/docs-html/current/ref/refse106.html#x216-23800016.2

Close,
https://www.freepascal.org/docs-html/current/ref/refse105.html#x215-23700016.1

Show quoted text 25 lines
Btw, the docs about Dotted units https://www.freepascal.org/docs-html/current/ref/refse107.html states: "When resolving symbols, unit scopes always take precedence over symbols inside units." IMO this is misleading. For example: uses foo; type TMy = class foo: TMy; procedure DoSome; end; procedure TMy.DoSome; begin foo:=nil; // nearest symbol wins foo.foo:=nil; // symbol inside unit wins end; The rule is more like this: If the first symbol of a dotted identifier is an unit in the uses section, the longest match wins.

Yes, it is somewhat like black magic :)

I will adapt it.

Michael.

Reply by email

Luca Olivetti
to Mattias Gaertner: On Wed, 17 May 2017 16:34:14 +0200 Fixed. The Lazarus devels only support the last two…

El 17/05/17 a les 16:48, Mattias Gaertner via Lazarus ha escrit:

Show quoted text 13 lines
On Wed, 17 May 2017 16:34:14 +0200 Luca Olivetti via Lazarus wrote: > [...] >> Is this version still supposed to work with fpc 2.6.4? > > OK, I see now that it doesn't build with fpc 2.6.4 :-( Fixed. The Lazarus devels only support the last two FPC releases, which means 3.0.0 and 3.0.2. But as long as FPC 2.6.4 only needs a few IFDEFs, patches are welcome.

Here's another one. Not sure it's correct but the dialog seems to be working fine

Index: components/codetools/ide/codyidentifiersdlg.pas =================================================================== --- components/codetools/ide/codyidentifiersdlg.pas (revision 54949) +++ components/codetools/ide/codyidentifiersdlg.pas (working copy) @@ -1366,7 +1366,7 @@

    NewY: integer;
    NewTopLine: integer;
    CurUnit: TUDUnit;

- MainPath: RawByteString;
+ MainPath: {$IF FPC_FULLVERSION < 30000} string {$ELSE} RawByteString {$ENDIF};

    function OpenDependency: boolean;
    // returns false to abort
Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Mattias Gaertner
to Luca Olivetti
Show quoted text 12 lines
[...] Index: components/codetools/ide/codyidentifiersdlg.pas =================================================================== --- components/codetools/ide/codyidentifiersdlg.pas (revision 54949) +++ components/codetools/ide/codyidentifiersdlg.pas (working copy) @@ -1366,7 +1366,7 @@ NewY: integer; NewTopLine: integer; CurUnit: TUDUnit; - MainPath: RawByteString; + MainPath: {$IF FPC_FULLVERSION < 30000} string {$ELSE} RawByteString {$ENDIF};

It must be string. Fixed.

Mattias

Reply by email

Ondrej Pokorny
to Mattias Gaertner
Mattias Gaertner wrote 16 lines
On Thu, 18 May 2017 16:57:40 +0200 Luca Olivetti via Lazarus wrote: > [...] > Index: components/codetools/ide/codyidentifiersdlg.pas > =================================================================== > --- components/codetools/ide/codyidentifiersdlg.pas (revision 54949) > +++ components/codetools/ide/codyidentifiersdlg.pas (working copy) > @@ -1366,7 +1366,7 @@ > NewY: integer; > NewTopLine: integer; > CurUnit: TUDUnit; > - MainPath: RawByteString; > + MainPath: {$IF FPC_FULLVERSION < 30000} string {$ELSE} RawByteString > {$ENDIF}; It must be string. Fixed.

IMO, we should create a new option for CodeTools to override string result types for Ctrl+Shift+C. It could be both easy to implement and easy to understand:

if the first argument is some kind of 8bit-string
(RawByteString/AnsiString/String), and the result is an 8bit string as well, use the argument string type as result type.

In this case

var
   S: string;
begin
   X := ExtractFilePath(S);
end;

should create

var
   S, X: string;
begin
   X := ExtractFilePath(S);
end;

and not (what is now)

var
   S: string;
   X: RawByteString;
begin
   X := ExtractFilePath(S);
end;

A lot of the above issues would never happen.

Ondrej

Reply by email

Mattias Gaertner
to Ondrej Pokorny
Show quoted text 36 lines
[...] IMO, we should create a new option for CodeTools to override string result types for Ctrl+Shift+C. It could be both easy to implement and easy to understand: if the first argument is some kind of 8bit-string (RawByteString/AnsiString/String), and the result is an 8bit string as well, use the argument string type as result type. In this case var S: string; begin X := ExtractFilePath(S); end; should create var S, X: string; begin X := ExtractFilePath(S); end; and not (what is now) var S: string; X: RawByteString; begin X := ExtractFilePath(S); end; A lot of the above issues would never happen.

True.

Mattias

Reply by email

Ondrej Pokorny
to Mattias Gaertner: On Thu, 18 May 2017 17:14:03 +0200 True. Mattias
Mattias Gaertner wrote 40 lines
On Thu, 18 May 2017 17:14:03 +0200 Ondrej Pokorny via Lazarus wrote: > [...] > IMO, we should create a new option for CodeTools to override string > result types for Ctrl+Shift+C. It could be both easy to implement and > easy to understand: > > if the first argument is some kind of 8bit-string > (RawByteString/AnsiString/String), and the result is an 8bit string as > well, use the argument string type as result type. > > In this case > > var > S: string; > begin > X := ExtractFilePath(S); > end; > > should create > > var > S, X: string; > begin > X := ExtractFilePath(S); > end; > > and not (what is now) > > var > S: string; > X: RawByteString; > begin > X := ExtractFilePath(S); > end; > > > A lot of the above issues would never happen. True.

Should I enable it by default?

Ondrej

Reply by email

Ondrej Pokorny
to Ondrej Pokorny
Ondrej Pokorny wrote
On 18.05.2017 17:20, Mattias Gaertner via Lazarus wrote: > True. Should I enable it by default?

I implemented it in r54976 and made it default because all 8-bit string types are equivalent for the compiler, so it makes perfect sense not to mix them; if I am not mistaken.

Ondrej

Reply by email

Werner Pamler
to Mattias Gaertner: The Lazarus team is glad to announce the first release candidate of Lazarus 1.8. This…

An issue with the installer for 1.8RC1 32bit Windows (I hope it has not been discussed already...): the checkbox to create a desktop icob is disabled.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170518/aab9dc02/attachment.html> -------------- next part --------------
A non-text attachment was scrubbed...
Name: kaghndbcbckilogh.png
Type: image/png
Size: 15652 bytes
Desc: not available
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20170518/aab9dc02/attachment.png>

Reply by email

Werner Pamler
to Werner Pamler: This is confusing: Are you using Laz trunk (v1.9) or 1.8RC1? I don't see an issue with…
Werner Pamler wrote 13 lines
Am 17.05.2017 um 23:08 schrieb Andreas Frieß via Lazarus: > I see with the last patches something goes wrong after r54880 with > frames and grid (specially TsWorkSheetGrid). > > Lazarus claims, the is no valid parent and didnot load and crash. It > is only happen if you use frames ? The trouble is, i can only > shutdown Lazarus with the Processmanger. > > Win10/64 with Lazarus 32/svn 1.9RC1 This is confusing: Are you using Laz trunk (v1.9) or 1.8RC1? I don't see an issue with WorksheetGrid on frames with Laz trunk. Can you post a demo?

Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has no parent" error with frames which had crept in, again...

Reply by email

Werner Pamler
to Werner Pamler: An issue with the installer for 1.8RC1 32bit Windows (I hope it has not been discussed…
Werner Pamler wrote
An issue with the installer for 1.8RC1 32bit Windows (I hope it has not been discussed already...): the checkbox to create a desktop icob is disabled.

This happens when "Create a seconary installation" is checked. And I looked at previous versions: all versions back to 1.2 have the same behavior (1.0 did not have the secondary installation option yet). So, I guess this is intended. But why should a secondardy installation be inhibited to create a desktop icon?

Reply by email

Martin Frb
to Werner Pamler
Werner Pamler wrote 10 lines
Am 18.05.2017 um 23:45 schrieb Werner Pamler via Lazarus: > An issue with the installer for 1.8RC1 32bit Windows (I hope it has > not been discussed already...): the checkbox to create a desktop icob > is disabled. This happens when "Create a seconary installation" is checked. And I looked at previous versions: all versions back to 1.2 have the same behavior (1.0 did not have the secondary installation option yet). So, I guess this is intended. But why should a secondardy installation be inhibited to create a desktop icon?

No idea (that is, I do not remember)

Probably to avoid overwriting the desktop icon of the primary install. So basically it would need implementation to create a meaningful icon. (If inno can be tweaked).
If so then it is a missing feature. patches welcome.

Reply by email

Andreas Frieß
to Werner Pamler: Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has no parent" error…
Werner Pamler wrote 18 lines
Am 18.05.2017 um 00:41 schrieb Werner Pamler via Lazarus: > Am 17.05.2017 um 23:08 schrieb Andreas Frieß via Lazarus: >> I see with the last patches something goes wrong after r54880 with >> frames and grid (specially TsWorkSheetGrid). >> >> Lazarus claims, the is no valid parent and didnot load and crash. It >> is only happen if you use frames ? The trouble is, i can only >> shutdown Lazarus with the Processmanger. >> >> Win10/64 with Lazarus 32/svn 1.9RC1 > > This is confusing: Are you using Laz trunk (v1.9) or 1.8RC1? I don't > see an issue with WorksheetGrid on frames with Laz trunk. Can you > post a demo? Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has no parent" error with frames which had crept in, again...

Werner thank you, i will make a try and i will look for the changes, because i found other (older) components they have some trouble with frames (DirectShow) too. Next week i can spend some time to check this with a fresh installation to avoid side effects.

Andreas

Reply by email

Luca Olivetti
to Werner Pamler: Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has no parent" error…

El 19/05/17 a les 01:30, Werner Pamler via Lazarus ha escrit:

Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has no parent" error with frames which had crept in, again...

I just discovered that with this revision OnSelectCell doesn't fire anymore. Not a biggie, I'm using OnClick instead, but it caught me by surprise.

Bye

Luca Olivetti Wetron Automation Technology http://www.wetron.es/ Tel. +34 93 5883004 (Ext.3010) Fax +34 93 5883007

Reply by email

Werner Pamler
to Luca Olivetti
Luca Olivetti wrote 8 lines
El 19/05/17 a les 01:30, Werner Pamler via Lazarus ha escrit: > Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has > no parent" error with frames which had crept in, again... I just discovered that with this revision OnSelectCell doesn't fire anymore. Not a biggie, I'm using OnClick instead, but it caught me by surprise.

Thanks for reporting. Should be fixed in r5905.

Reply by email

Werner Pamler
to Mattias Gaertner: The Lazarus team is glad to announce the first release candidate of Lazarus 1.8. This…

Forum user "handoko" reports some issues with RC1 in http://forum.lazarus.freepascal.org/index.php/topic,37022.msg247603.html#msg247603 (and
http://forum.lazarus.freepascal.org/index.php/topic,36907.msg247608.html#msg247608), among them the "double-keyboard" error under Ubuntu again. Maybe someone more knowledgeable than me can have a look at this.

Reply by email

Andreas
to Andreas Frieß: Werner thank you, i will make a try and i will look for the changes, because i found…

Lazarus 1.8.0RC1 r55107 FPC 3.1.1 i386-win32-win32/win64 with fpspreadsheet r5910 its ok.

THX 4 your help and your time

Gesendet von Mail für Windows 10

Von: Andreas Frieß via Lazarus
Gesendet: Samstag, 27. Mai 2017 11:29
An:
Betreff: Re: [Lazarus] Lazarus Release Candidate 1 of 1.8.0

Werner Pamler wrote 18 lines
Am 18.05.2017 um 00:41 schrieb Werner Pamler via Lazarus: > Am 17.05.2017 um 23:08 schrieb Andreas Frieß via Lazarus: >> I see with the last patches something goes wrong after r54880 with >> frames and grid (specially TsWorkSheetGrid). >> >> Lazarus claims, the is no valid parent and didnot load and crash. It >> is only happen if you use frames ? The trouble is, i can only >> shutdown Lazarus with the Processmanger. >> >> Win10/64 with Lazarus 32/svn 1.9RC1 > > This is confusing: Are you using Laz trunk (v1.9) or 1.8RC1? I don't > see an issue with WorksheetGrid on frames with Laz trunk. Can you > post a demo? Andreas, could you try r5868 of fpspreadsheet? I fixed a "Window has no parent" error with frames which had crept in, again...

Werner thank you, i will make a try and i will look for the changes, because i found other (older) components they have some trouble with frames (DirectShow) too. Next week i can spend some time to check this with a fresh installation to avoid side effects.

Andreas

Reply by email