Re: [MSEide-MSEgui-talk] msedbusinterface and fpc 3.3.1 trunk
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
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
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
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
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
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
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