Hi Carl, Now that the fix is in AOO41X I will prepare a new build (for Windows).
Maybe Lucien can test/confirm the issue is solved? @Lucien: What OS do you use? Regards, Matthias Am 07.06.22 um 23:23 schrieb Carl Marcum: > Hi Matthias, > > On 6/7/22 6:53 AM, Matthias Seidel wrote: >> Hi Carl, >> >> Am 07.06.22 um 00:59 schrieb Carl Marcum: >>> Hi Lucien, >>> >>> On 6/6/22 12:51 PM, Lucien Mathay wrote: >>>> Thank you Regina, but >>>> >>>> if I add an 'endif' at the end of the line >>>> ( " if a = b then a=1 Else a=2 endif 'test "), >>>> the compiler fails with the message "Syntax error : unexpectes symbol >>>> : End If". >>>> >>>> Furthermore, the book from "OpenOffice .org Macros OoOffice et >>>> Apis" from Bernard Marcelly and Laurent Goddard states p.118 : >>>> >>>> "Lorsqu’une seule instruction suffit dans la partie Then et dans la >>>> partie Else, la séquence peut s’écrire sur une seule ligne : >>>> If expr1 Then instruction1v Else instruction1f >>>> Notez l’absence du End If dans cette forme simplifiée." >>>> which means, translated : >>>> >>>> "When only one instruction is used in the section Then and in the >>>> section Else, the sequence can be written on one single line : >>>> If expr1 Then instruction1v Else instruction1f >>>> Please note the absence of End If in this simplified usage" >>>> >>>> Therefore I still consider this as a bug. >>> I believe you are correct. >>> >>> In my recent work on making the trunk test suites standalone to run >>> against other branches like AOO41X I discovered some other bug fixes >>> that were applied to trunk and AOO42X but never back ported to AOO41X. >>> >>> Two examples I put in a PR-150 [1]. One of which related to variable >>> names in single-line if statements. >>> I tested your example against that build but it isn't fixed by it but >>> I believe I found the patch that fixed your bug in trunk [2]. >>> Issue 126272 [3] is listed in Bugzilla with a target milestone of 4.2. >>> I think this needs a more general discussion on dev@ about how much we >>> should change API's in 4.1.X. >>> Which I intended to do anyway before merging my PR-150. >>> >>> Thanks for pointing this out! >>> >>> [1] https://github.com/apache/openoffice/pull/150 >>> [2] >>> https://github.com/apache/openoffice/commit/07396187f6055b1e7cffa86f38cc88b274dfb1d6 >>> >>> [3] https://bz.apache.org/ooo/show_bug.cgi?id=126272 >> I think this fix [2] should be cherry-picked to AOO41X. >> >> BTW: The target milestone 4.2.0 was trunk at that time. Later we made >> trunk 4.5.0 and branched 4.2.0, but the milestones were not updated. > > Yes, I just want to make sure it wasn't intentional to leave this one > and a few other changes the the Basic macros out of the 4.1 line due > to not wanting to change API or code behavior. > I will create a PR for it or maybe just add it to the other one I've > got open since they are all small and bring a discussion on dev@. > > Then if we agree it's okay I'll pull them in. > > Best regards, > Carl >> >> Regards, >> >> Matthias >> >>> Best regards, >>> Carl >>>> With kind regards, >>>> Lucien >>>> >>>> Le 6/06/22 à 13:32, Regina Henschel a écrit : >>>>> Hi Lucien, >>>>> >>>>> Lucien Mathay schrieb am 06.06.2022 um 10:42: >>>>>> Hello, >>>>>> >>>>>> I would like to report the following bug : in the macros when a >>>>>> line containing "if ... then ... else" is followed by a comment on >>>>>> the same line, the compiler fails. >>>>>> >>>>>> Example : >>>>>> >>>>>> Function test() >>>>>> dim a as long, b as long >>>>>> a=0: b=0 >>>>>> if a = b then a=1 else a=2 'test >>>>>> b=1 >>>>>> call msgbox b >>>>>> End Function >>>>> The if-statement misses endif. >>>>> >>>>> Kind regards, >>>>> Regina >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: qa-unsubscr...@openoffice.apache.org >>> For additional commands, e-mail: qa-h...@openoffice.apache.org >>> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: qa-unsubscr...@openoffice.apache.org > For additional commands, e-mail: qa-h...@openoffice.apache.org >
smime.p7s
Description: S/MIME Cryptographic Signature