Re: [MSEide-MSEgui-talk] Notes & Warnings

2023-03-01 Thread Fred van Stappen
Hello Sieghard.

Thanks for your kind words.

> I just noticed two occurences of an insertion,
>"// result:= stockobjects.extended [index];
>   result:=   lang_extended [ord (index)];"
> in unit "msestockobjects.pas". As one of my goals was to get rid of
> the direct use of the "lang_"-arrays, I wonder why these might be required?

I did this because if I keep your modif, when trying to delete a line from a 
grid, the application crash.

You may test it with ideU, in Project Option form, add a line in a grid (for 
example in Tab "make", sub-tag "Directories") and then try to delete this line.
When deleting a line, the stockobjects.extended [index] with "Do you want to 
delete the line?" is called and make crash the application.

It is not the case when using lang_extended [ord (index)].

Fre;D



De : Sieghard via mseide-msegui-talk 
Envoyé : mardi 28 février 2023 22:20
À : mseide-msegui-talk@lists.sourceforge.net 

Cc : Sieghard 
Objet : Re: [MSEide-MSEgui-talk] Notes & Warnings

Hallo Fred,

vous ecrit au Tue, 28 Feb 2023 00:10:25 +:

> Dear Sieghard your mseide-msegui-shadow fixes are in GitHub via
> commit 1f52efa. I tested it and all is ok, many thanks for all those
> warnings away.

Thanks, not worth mentioning, surely not so prominently. This was just
out of anger about so many unneccessary warnings, prone to drown out
those that really _are_ relevant and need to be fixed.

> I have seen you added /db/FieldTypeError.pas to manage error with db
> field type errors and updated all units to use it. Nice and very
> usefull.

Well, yes, indeed... I really had to check and find out what that means.
And, indeed, that's a small hack I introduced to have better control
over database operations. I'm certain I DID miss a number of instances
where it might be of use, and probabely left out (of ignorance) a few
places which might also warrant a hint. Fine when you deem it useful.

> Thanks again and have perfect days.

Thank you for your good work, it's fine to get notice from you again.
I was already about asking myself whether mseide-msegui might have
gone under already. Of course, I got me your latest version and looked
for the modifications, finding that I already had introduced most of
them in my working copy.
I just noticed two occurences of an insertion,
"// result:= stockobjects.extended [index];
   result:=   lang_extended [ord (index)];"
in unit "msestockobjects.pas". As one of my goals was to get rid of
the direct use of the "lang_"-arrays, I wonder why these might be
required?

But anyway, I'm really glad to see that work on mseide-msegui is still
in progress and it is active and evolving.

Again, thank you for your good work, I wish you well and good luck!

--
(Weitergabe von Adressdaten, Telefonnummern u.ä. ohne Zustimmung
nicht gestattet, ebenso Zusendung von Werbung oder ähnlichem)
---
Mit freundlichen Grüßen, S. Schicktanz
---



___
mseide-msegui-talk mailing list
mseide-msegui-talk@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mseide-msegui-talk
___
mseide-msegui-talk mailing list
mseide-msegui-talk@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mseide-msegui-talk


Re: [MSEide-MSEgui-talk] Notes & Warnings

2023-03-01 Thread Sieghard via mseide-msegui-talk
Hallo Fred,

vous ecrit au Tue, 28 Feb 2023 00:10:25 +:

> Dear Sieghard your mseide-msegui-shadow fixes are in GitHub via
> commit 1f52efa. I tested it and all is ok, many thanks for all those
> warnings away.

Thanks, not worth mentioning, surely not so prominently. This was just
out of anger about so many unneccessary warnings, prone to drown out
those that really _are_ relevant and need to be fixed.

> I have seen you added /db/FieldTypeError.pas to manage error with db
> field type errors and updated all units to use it. Nice and very
> usefull.

Well, yes, indeed... I really had to check and find out what that means.
And, indeed, that's a small hack I introduced to have better control
over database operations. I'm certain I DID miss a number of instances
where it might be of use, and probabely left out (of ignorance) a few
places which might also warrant a hint. Fine when you deem it useful.

> Thanks again and have perfect days.

Thank you for your good work, it's fine to get notice from you again.
I was already about asking myself whether mseide-msegui might have
gone under already. Of course, I got me your latest version and looked
for the modifications, finding that I already had introduced most of
them in my working copy.
I just noticed two occurences of an insertion,
"// result:= stockobjects.extended [index];
   result:=   lang_extended [ord (index)];"
in unit "msestockobjects.pas". As one of my goals was to get rid of
the direct use of the "lang_"-arrays, I wonder why these might be
required?

But anyway, I'm really glad to see that work on mseide-msegui is still
in progress and it is active and evolving.

Again, thank you for your good work, I wish you well and good luck!

-- 
(Weitergabe von Adressdaten, Telefonnummern u.ä. ohne Zustimmung
nicht gestattet, ebenso Zusendung von Werbung oder ähnlichem)
---
Mit freundlichen Grüßen, S. Schicktanz
---



___
mseide-msegui-talk mailing list
mseide-msegui-talk@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mseide-msegui-talk