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. 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 >
smime.p7s
Description: S/MIME Cryptographic Signature