issue-#132 Unwanted characters in the definition... Refs:#19

Post Reply
User avatar
DGDanforth
Posts: 1061
Joined: Tue Sep 17, 2013 1:16 am
Location: Palo Alto, California, USA
Contact:

Re: issue-#132 Unwanted characters in the definition... Refs

Post by DGDanforth »

I can verify that

Code: Select all

DEFINITION Foo;

	TYPE
		Βασε = EXTENSIBLE RECORD  END;

		Εχτ = RECORD (Βασε 00X 懨 00X  00X ) END;

	PROCEDURE Do;

END Foo.
It appears there are 4 characters added.
-Doug
Zinn
Posts: 476
Joined: Tue Mar 25, 2014 5:56 pm
Location: Frankfurt am Main
Contact:

Re: issue-#132 Unwanted characters in the definition... Refs

Post by Zinn »

It is a very strange error.
Change in the PROCEDURE ProperString (s: ARRAY OF SHORTCHAR); of MODULE DevBrowser;
the line
i := 0; WHILE (s # 0X) & (s # "^") DO global.out.WriteChar(str); INC(i) END
to
i := 0; WHILE (i < LEN(str$)) & (s # 0X) & (s # "^") DO global.out.WriteChar(str); INC(i) END
and it works.
Why does not work the compare s # 0X ?
- Helmut
User avatar
DGDanforth
Posts: 1061
Joined: Tue Sep 17, 2013 1:16 am
Location: Palo Alto, California, USA
Contact:

Re: issue-#132 Unwanted characters in the definition... Refs

Post by DGDanforth »

Partial answer.
SHORTCHAR can not hold a Uni char.
Zinn
Posts: 476
Joined: Tue Mar 25, 2014 5:56 pm
Location: Frankfurt am Main
Contact:

Re: issue-#132 Unwanted characters in the definition... Refs

Post by Zinn »

Now I see the error. I must be blind.
The correct line is
i := 0; WHILE (str # 0X) & (str # "^") DO global.out.WriteChar(str); INC(i) END
Helmut
Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

Re: issue-#132 Unwanted characters in the definition... Refs

Post by Ivan Denisov »

User avatar
Josef Templ
Posts: 2047
Joined: Tue Sep 17, 2013 6:50 am

Re: issue-#132 Unwanted characters in the definition... Refs

Post by Josef Templ »

The fix looks fine to me.

Note: We also need to update appbuild/AppVersion.txt to "1.7.1-a1".

- Josef
Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

Re: issue-#132 Unwanted characters in the definition... Refs

Post by Ivan Denisov »

I have changed the app version

http://blackboxframework.org/unstable/i ... a1.698.zip

Doug, this issue is tested and ready for voting.
User avatar
Josef Templ
Posts: 2047
Joined: Tue Sep 17, 2013 6:50 am

Re: issue-#132 Unwanted characters in the definition... Refs

Post by Josef Templ »

agree
User avatar
DGDanforth
Posts: 1061
Joined: Tue Sep 17, 2013 1:16 am
Location: Palo Alto, California, USA
Contact:

Re: issue-#132 Unwanted characters in the definition... Refs

Post by DGDanforth »

Vote has been created.
Post Reply