Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[code dz]

I dont remember exactly . but i think Its about range checking .
On 9 Mar 2020 20:29, "fredvs"  wrote:

> Re-hello Code DZ.
>
> > I tried fixing these warning but i lost .
>
> What makes you stop, what kind of warnings?
>
> Fre;D
>
>
>
> --
> Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/
>
>
> ___
> 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] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Hello Seighard.

Thanks for that clear explanations.

By the way, I did check in all the msegui sources if there was, by bad
chance, a other case with omitted parenthesis in "is" operator combined with
"and".

In all the code written by Martin, when "is" is used with "and", the
parenthesis are used.

So, maybe, it was indeed a bug (or omission) in that unique case in all
msegui code.

Fre;D



--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[Sieghard]

Hello fredvs,

you wrote on Mon, 9 Mar 2020 02:00:30 -0700 (MST):

> > I'd rather suggest parentheizing the first partial expression, like so:
> >  if (fld1 is tblobfield) and (self.fblobintf <> nil) then begin   
> 
> Yes, it is exactly what was proposed in fpc mailing list.
> 
> It is also what I did commit in mse-org/mseide_msegui

Good. Relying on the operator precedence can often cause problems, and it
becomes completely unreliable when diverse types are involved.
Usually, I try to avoid the ambiguity of such constructs by explicitely
parenthesizing them. This is mainly because I often have to change between
different languages, and thus different operator hierarchies. About the
only safe precedence rules are those of school mathematics - i.e.
multiply/divide comes before add/subtract - but anything else, even
handling of booleans, can vary.
I'd recommend copiously paranthesizing complex expressions as a standard,
and parentesizing partial expressions resulting in a different type from
the operands specifically.

(Of course, in many such cases, it might be possible for the compiler to
resolve an ambigous construct by contemplating the type structure of the
expression, but there _will_ be cases where that's _not_ what the writer
intended - so at least a warning will have to be issued and an explanation
of the assumed coercion given, As that is not a trivial thing, it seems
most implementations "play it dumb" and only perform minimal processing.)

-- 
-- 
(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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Re-hello Code DZ.

> I tried fixing these warning but i lost . 

What makes you stop, what kind of warnings?

Fre;D



--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Hello Code dz

>  i think msegui should stay with fpc3.0.4 until finding the fix

Yes, of course, fpc 3.0.4 is the "official" last release of fpc.

I do regularly check the new features of fpc 3.3.1 trunk, this to not be too
far of what appends.
(And I agree also to check if ,by chance, they fixed backward compatibility
with fpc 3.0.4).

After deep check, fpc 3.2.0 beta is fully compatible with msegui.

But yes, I would not recommend at the moment to people to use fpc 3.3.1
trunk in production, even for no msegui project.

Fre;D






--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[Graeme Geldenhuys]

On 09/03/2020 9:00 am, fredvs wrote:
>> I'd rather suggest parentheizing the first partial expression, like so:
>>  if (fld1 is tblobfield) and (self.fblobintf <> nil) then begin 
> Yes, it is exactly what was proposed in fpc mailing list.

Nice, good catch Sieghard.

Regards,
  Graeme


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[code dz]

Hi fred. I tried fixing these warning but i lost .  i think msegui should
stay with fpc3.0.4 until finding the fix .
On 8 Mar 2020 14:11, "fredvs"  wrote:

> Re-hello.
>
> OK, I did apply the Graemes's fix: adding {$interfaces corba}, just after
> unit msedbusinterface.
>
> This fixes the error:
>
> "Error: The interface type of the forward
> declaration and the declared interface type do not match for interface
> idbusservice "
>
> Nice but now there is a new other error:
>
> msesqlquery.pas(1719,30) Error: Operator is not overloaded: "Class Of
> TBlobField" and "Boolean"
>
> (G + mama-mia + w** + censured + ...)
>
> Here the line with problems:
>
>  --->  if fld1 is tblobfield and (self.fblobintf <> nil) then begin
>
> Changing, as workaround with:
>
> --->   if (self.fblobintf <> nil) then begin
>
> Makes fpc 3.3.1 trunk happy.
>
> Do you know what is the new equivalent for:
> ---> if fld1 is tblobfield // boolean no more permitted
>
> ?
>
> Of course, I did not commit anything yet, I wait for your comments.
>
> Fre;D
>
>
>
> --
> Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/
>
>
> ___
> 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] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Hello Sieghard.

> I'd rather suggest parentheizing the first partial expression, like so:
>  if (fld1 is tblobfield) and (self.fblobintf <> nil) then begin 

Yes, it is exactly what was proposed in fpc mailing list.

It is also what I did commit in mse-org/mseide_msegui

https://github.com/mse-org/mseide-msegui/commit/d9d2e4254d12633ea2a3b38d5060cb0060d4e8c6

Thanks.

Fre;D






--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[Sieghard]

Hello fredvs,

you wrote on Sun, 8 Mar 2020 06:10:57 -0700 (MST):

> Here the line with problems:
> 
>  --->  if fld1 is tblobfield and (self.fblobintf <> nil) then begin   
> 
> Changing, as workaround with:
> 
> --->   if (self.fblobintf <> nil) then begin   
> 
> Makes fpc 3.3.1 trunk happy.

But loses the plausibility control.
I'd rather suggest parentheizing the first partial expression, like so:

  if (fld1 is tblobfield) and (self.fblobintf <> nil) then begin

This way, there's no ambiguity about operator application, and both partial
expressions evaluate to type "boolean", so the "and" will be satisfied and
evaluated correctly.
If that's a solution to the problem, the reason was an ambiguity in
operator precedence - "and" regarded higher then "is".

-- 
-- 
(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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[Roland Chastain]

Hello!

Well done Fred.




--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Hello.

With my fpc friends, we fixed the problem:

fpc-topic

  

So, it was committed in
https://github.com/mse-org/mseide-msegui/commits/master

Fre;D
, 



--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Re-hello.

OK, I did apply the Graemes's fix: adding {$interfaces corba}, just after
unit msedbusinterface.

This fixes the error:

"Error: The interface type of the forward
declaration and the declared interface type do not match for interface
idbusservice "

Nice but now there is a new other error:

msesqlquery.pas(1719,30) Error: Operator is not overloaded: "Class Of
TBlobField" and "Boolean"

(G + mama-mia + w** + censured + ...)

Here the line with problems:

 --->  if fld1 is tblobfield and (self.fblobintf <> nil) then begin 

Changing, as workaround with:

--->   if (self.fblobintf <> nil) then begin 

Makes fpc 3.3.1 trunk happy.

Do you know what is the new equivalent for: 
---> if fld1 is tblobfield // boolean no more permitted

?

Of course, I did not commit anything yet, I wait for your comments.

Fre;D



--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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

[MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk

[fredvs]

Hello.

I just have try to compile mseide with last fpc trunk 3.3.1 of 08-03-2020.

There is that error message:

msedbusinterface.pas(224,5) Error: The interface type of the forward
declaration and the declared interface type do not match for interface
idbusservice
msedbusinterface.pas(564,1) Fatal: There were 1 errors compiling module,
stopping
Fatal: Compilation aborted

Any idea what could be the cause?

Thanks.

Fre;D



--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/


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