[Lazarus] Wish about Lazarus readme

2017-12-09 Thread AlexeyT via Lazarus

On 10.12.2017 01:50, Vojtěch Čihák via Lazarus wrote:
I didn't know about Ally. 


Wish for contributions.txt of Lazarus: separate section for picture 
painters: separate it with caption "Graphic artists" and write like this:


Ally (IDE pallette icons)

FTurtle (IDE command icons)




--
Regards,
Alexey

--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] Note to new high-DPI icons

2017-12-09 Thread Vojtěch Čihák via Lazarus

I mean dark OS theme. See attached screeenshot. It was blue before, it can be 
found on the wiki: 
http://wiki.freepascal.org/Lazarus_Tutorial#The_Component_Palette
Such blueish theme is IMO unusual, the most common themes are light and dark 
grey (or black), sometimes with only a small saturation.
 
I didn't know about Ally. I saw issues with icon-patches on bugtracker opened 
by FTurtle, I thought he is author.
 
V.
__

Od: Werner Pamler via Lazarus 
Komu: lazarus@lists.lazarus-ide.org
Datum: 09.12.2017 22:24
Předmět: Re: [Lazarus] Note to new high-DPI icons


Am 09.12.2017 um 16:18 schrieb Vojtěch Čihák via Lazarus:
> Hi,
>   
> I tried to run Lazarus with dark theme and icons for TLabel and TStaticText are not visible. I propose to use that blue color used with the old icons instead of black.
>   
> V.
>   
> PS: I have no direct contact to author of icons, FTurtle, so I sent it to Lazarus ML.


Which "dark theme"? Is Lazarus themed? Or do you mean OS theme? What if 
somebody selects a blue-ish theme?


FTurtle is not responsible for the palette icons; those were drawn by 
Ally, a user of the German Lazarus forum.

--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus 


-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] Note to new high-DPI icons

2017-12-09 Thread Werner Pamler via Lazarus

Am 09.12.2017 um 16:18 schrieb Vojtěch Čihák via Lazarus:

Hi,
  
I tried to run Lazarus with dark theme and icons for TLabel and TStaticText are not visible. I propose to use that blue color used with the old icons instead of black.
  
V.
  
PS: I have no direct contact to author of icons, FTurtle, so I sent it to Lazarus ML.


Which "dark theme"? Is Lazarus themed? Or do you mean OS theme? What if 
somebody selects a blue-ish theme?


FTurtle is not responsible for the palette icons; those were drawn by 
Ally, a user of the German Lazarus forum.

--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] CHM help package

2017-12-09 Thread Graeme Geldenhuys via Lazarus

On 2017-12-09 17:33, Marco van de Voort via Lazarus wrote:

yada,yada,yada git plug


Oh just fuck off Marco!  Not everything I say is meant to piss you off 
or even directed at you. Please add me to your email ignore list and be 
done with it.


Regards,
  Graeme

--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] CHM help package

2017-12-09 Thread Marco van de Voort via Lazarus
On Sat, Dec 09, 2017 at 04:49:56PM +, Graeme Geldenhuys via Lazarus wrote:
> On 2017-12-08 12:23, Sergey Bodrov via Lazarus wrote:
> > There is too many changes in many files, and that can't be posted as simple
> > diff patch.
> 
> Use the Git clone (mirror) of the Lazarus (svn) repository. 

yada,yada,yada git plug

I assume you haven't read the message at all, since he did all that at
github, but ran code formatters over it. So that makes diff, any diff a
problem. 

And pull requests also don't solve that.

-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] CHM help package

2017-12-09 Thread Graeme Geldenhuys via Lazarus

On 2017-12-08 12:23, Sergey Bodrov via Lazarus wrote:

There is too many changes in many files, and that can't be posted as simple
diff patch.


Use the Git clone (mirror) of the Lazarus (svn) repository. Git allows 
you to make local branch and local commits - invaluable in today's 
development (personal opinion, but agreed by many). You can then send 
pull requests - many Lazarus SVN developers seem to be willing to accept 
that, but confirm with the maintainers of that section of the project 
first. Alternative, you can ask git to generate a patch set by simply 
giving the starting and ending commit SHA1 values. It will generate many 
patch files in the order that they need to be applied.


  Lazarus Git mirror: https://github.com/graemeg/lazarus.git

To clone from the command line:

   git clone https://github.com/graemeg/lazarus.git

The above command only needs to be done once.

   git checkout -b my_features_branch
   // make your commits here

The above creates your own local branch were you place your local 
commits. Good advice - commit often!


   git format-patch -o /tmp/ origin/master..HEAD

The above will generate a patch set and output all the *.patch files 
into the /tmp/ directory.



Regards,
  Graeme

--
fpGUI Toolkit - a cross-platform GUI toolkit using Free Pascal
http://fpgui.sourceforge.net/

My public PGP key:  http://tinyurl.com/graeme-pgp
--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


[Lazarus] Note to new high-DPI icons

2017-12-09 Thread Vojtěch Čihák via Lazarus
Hi,
 
I tried to run Lazarus with dark theme and icons for TLabel and TStaticText are 
not visible. I propose to use that blue color used with the old icons instead 
of black.
 
V.
 
PS: I have no direct contact to author of icons, FTurtle, so I sent it to 
Lazarus ML.  
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] CHM help package

2017-12-09 Thread Marco van de Voort via Lazarus
On Fri, Dec 08, 2017 at 03:23:06PM +0300, Sergey Bodrov via Lazarus wrote:
> There is too many changes in many files, and that can't be posted as simple
> diff patch.

Then layer them. Make patches for the most essential bits, and then reallign
your sources.

> 1. FPC package 'chm' contain file fasthtmlparser.pas, that used in some
> non-CHM packages and projects. May be better move that file to some generic
> package - fcl-base, for example?

Afaik this was meant as a copy of that parser dedicated for chm that could
evolve with the package.  So if you need it more general, whip up a suitable
general version.
 
> 2. FPC package 'chm' contain LZX compression routines in some files
> - paslzx.pas, paslzxcomp.pas, paslznonslide.pas, lzxcompressthread.pas.
> They not depends on other files in 'chm' and can be maintained as separate
> package.

What does that make possible that currently is not possible? Do we really
have to spin each unit into a separate package? I don't see the point.

Note also that iirc CHM lzx and e.g. CAB lzx vary in some ways. (headers?)

But even bypassing that, what is the problem of having to instal 10 extra
units (150kb) to make a CAB decompressor?

-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


[Lazarus] fgl unit, TFPSList needs DeleteRange

2017-12-09 Thread Alexey via Lazarus
I have 1M lines file. CudaText cannot delete lines fast [selected 800K 
lines]. FPSList has only Delete for one item.


Need DeleteRange.

--
Regards,
Alexey

--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] Lazarus Release 1.8

2017-12-09 Thread Luca Olivetti via Lazarus

El 09/12/17 a les 13:23, Juha Manninen via Lazarus ha escrit:


On Sat, Dec 9, 2017 at 12:57 PM, Luca Olivetti via Lazarus
 wrote:

I know I'll have to upgrade to fpc 3 someday, but what worries me about it
is my (and other 3rd party units) use of ansistrings as binary buffers.


The automatic conversions by FPC 3 could indeed mess things up.
However you should change such binary buffers to use TBytes or similar
in any case.


I think I had this discussion before: I'm not going to use TBytes until 
it offers the same convenience as an ansistring (i.e. easy 
concatenation, insertion and deletion, not to mention search and 
replace, etc.).
Besides, some of it is in 3rd party libraries (but I think they 
explicitly use ansistring, not the generic string type).



There have always been warnings against using String type
for binary data. In future your code may use $mode DelphiUnicode and
it breaks for sure.


There weren't when I started using them in delphi 20 years ago (with no 
dynamic arrays in sight).


Bye
--
Luca Olivetti
Wetron Automation Technology http://www.wetron.es/
Tel. +34 93 5883004 (Ext.3010)  Fax +34 93 5883007
--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] Lazarus Release 1.8

2017-12-09 Thread Juha Manninen via Lazarus
On Sat, Dec 9, 2017 at 1:41 AM, Werner Pamler via Lazarus
 wrote:
> Tested on Win32 where it fails in interfaces\win32\win32int because unit
> "character" is not found. It is in rtl\objpas of fpc3.04, but not of fpc
> 2.6.4, it came in with fpc 3.0.0

Damn, it is added by me.
It is used only for function TCharacter.IsSurrogate(). Do you know a
replacement function without copying the whole unit from FPC 3.0?
LazUTF16 only has:
 function IsUTF16CharValid(AChar, ANextChar: WideChar): Boolean;
A fix could be committed to the 1.8 fixes branch as a special case
If no easy fix is found then I could add:
  if {$IF 
FPC_FULLVERSION<3}False{$ELSE}TCharacter.IsSurrogate(WideChar(OrgCharCode)){$ENDIF}
then
and the code works with FPC 2.6.4 as it did before my changes.

The call to TCharacter.IsSurrogate() fixes this issue only partly :
 https://bugs.freepascal.org/view.php?id=32101
UTF8KeyPress is still not called. Somebody with better knowledge of
LCL-Win32 should fix it.


On Sat, Dec 9, 2017 at 12:57 PM, Luca Olivetti via Lazarus
 wrote:
> I know I'll have to upgrade to fpc 3 someday, but what worries me about it
> is my (and other 3rd party units) use of ansistrings as binary buffers.

The automatic conversions by FPC 3 could indeed mess things up.
However you should change such binary buffers to use TBytes or similar
in any case. There have always been warnings against using String type
for binary data. In future your code may use $mode DelphiUnicode and
it breaks for sure.
Maybe switching to FPC 3 is the right moment to fix the buffers, too.

Juha
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] Lazarus Release 1.8

2017-12-09 Thread Luca Olivetti via Lazarus

El 09/12/17 a les 00:41, Werner Pamler via Lazarus ha escrit:

Am 08.12.2017 um 22:32 schrieb Juha Manninen via Lazarus:

How does it fail? Lazarus 1.8 was meant to still compile with FPC 2.6.4.
However with Lazarus trunk I would be happy to go ahead and forget 2.6.4.
If the fix for Lazarus 1.8 is easy, please upload a patch.


Tested on Win32 where it fails in interfaces\win32\win32int because unit 
"character" is not found. It is in rtl\objpas of fpc3.04, but not of fpc 
2.6.4, it came in with fpc 3.0.0


Yes, my dirty hack was to copy character.pas from fpc-3.0.0 (and all 
other needed units[*]) to interfaces\win32.
I also add to edit character.pas to add the missing resource strings and 
the definition of ENoConstructException



--- fpc-3.0.0/rtl/objpas/character.pas  2013-08-20 06:55:05.0 +0200
+++ laz_1_8/lcl/interfaces/win32/character.pas  2017-12-09 
11:51:22.606416879 +0100

@@ -195,6 +195,19 @@
   SysUtils,
   RtlConsts;

+ResourceString
+  SStringIndexOutOfRange= 'String index %d out of range [1 - %d]';
+  SHighSurrogateOutOfRange  = 'High surrogate $%x out of range 
[$D800 - $DBFF]';
+  SLowSurrogateOutOfRange   = 'Low surrogate $%x out of range 
[$DC00 - $DFFF]';
+  SInvalidUTF32Char = 'Invalid UTF32 character $%x. Valid 
UTF32 character must be in range [$0 - $10] except surrogate range 
[$D800-$DFFF]';
+  SInvalidHighSurrogate = 'Invalid high surrogate at index %d. 
High surrogate must be followed by a low surrogate pair';

+  SInvalidUnicodeCodePointSequence = 'Invalid unicode code point sequence';
+  SClassCantBeConstructed   = 'Class %s can not be constructed';
+
+type
+
+  ENoConstructException = class(Exception);
+
 const
   LETTER_CATEGORIES = [
 TUnicodeCategory.ucUppercaseLetter, 
TUnicodeCategory.ucLowercaseLetter,



[*] the complete list of missing units is

?   character.pas
?   unicodedata.inc
?   unicodedata.pas
?   unicodedata_be.inc
?   unicodedata_le.inc
?   unicodenumtable.pas
?   weight_derivation.inc


I know I'll have to upgrade to fpc 3 someday, but what worries me about 
it is my (and other 3rd party units) use of ansistrings as binary 
buffers. I did some quick tests some time ago an nothing seemed broken, 
but I'm somewhat reluctant since my software has to run unattended 24/7 
(or more like 24/5).
OTOH I can always keep a copy of fpc-2.6.4 and lazarus 1.6.4 recompile 
if something goes wrong.



Bye
--
Luca Olivetti
Wetron Automation Technology http://www.wetron.es/
Tel. +34 93 5883004 (Ext.3010)  Fax +34 93 5883007
--
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] Some trunk changes make CudaText crash

2017-12-09 Thread Juha Manninen via Lazarus
On Sat, Dec 9, 2017 at 9:14 AM, Alexey via Lazarus
 wrote:
> I found the reason, it's my bug with generic lists.

Ok. In future you can build the whole IDE with debug flags, for
example with the "Debug IDE" profile in Configure Build Lazarus
window.
Packages like LCL and LazUtils also get the same debug flags.

Juha
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus