Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

/enforceself compiler option is being ignored #1503

Closed
cpyrgas opened this issue Jun 30, 2024 · 1 comment
Closed

/enforceself compiler option is being ignored #1503

cpyrgas opened this issue Jun 30, 2024 · 1 comment
Assignees
@RobertvanderHulst
Labels
bug Compiler
Milestone
2.x

Comments

@cpyrgas
Copy link

cpyrgas commented Jun 30, 2024 •
edited
Loading

Following code does not produce any compiler error about the missing SELF keywords:

#pragma options (enforceself, on)

CLASS TestClass
	PROTECT n AS INT
	EXPORT o AS OBJECT
	CONSTRUCTOR()
		? o == NULL
	RETURN
	
	METHOD Test() AS VOID
		n := 1
		? n
		? TestProperty
	RETURN
	PROPERTY TestProperty AS INT GET n
END CLASS
cpyrgas pushed a commit that referenced this issue Jun 30, 2024
[Compiler tests] Added C911 for #1503
8ec9425
RobertvanderHulst added a commit that referenced this issue Jul 1, 2024
@RobertvanderHulst
Feature/x sharp compiler (#1505)
1c6c7ad 
* [Compiler tests] Added C911 for #1503

* [docs] processed changes from #1504

---------

Co-authored-by: cpyrgas <chris@xsharp.eu>
@RobertvanderHulst RobertvanderHulst self-assigned this Jul 2, 2024
@RobertvanderHulst RobertvanderHulst added this to the 2.x milestone Jul 2, 2024
RobertvanderHulst added a commit that referenced this issue Jul 2, 2024
@RobertvanderHulst
[Compiler] Fixes C911 and #1503
e20c905
RobertvanderHulst added a commit that referenced this issue Jul 4, 2024
@RobertvanderHulst
[Compiler] Additonal change for the /enforceself issue because it was…
2e8698a 
… generating false reports. #1503
RobertvanderHulst added a commit that referenced this issue Jul 4, 2024
@RobertvanderHulst
Feature/x sharp compiler (#1511)
5e1b0b5 
* [doc] Added topics

* [Compiler] Fixes C911 and #1503

* [Compiler] Fix for #1419. There is a new error XS9127 for this with the message:
Type '{0}' cannot have an overloaded method '{1}' because one of the overloads has the CLIPPER calling convention.

* [Compiler] Fix for #1483

* [Tests] Added test for #1487

* [Compiler] Additonal change for the /enforceself issue because it was generating false reports. #1503
RobertvanderHulst added a commit that referenced this issue Jul 4, 2024
@RobertvanderHulst
[Compiler] Additional change for #1503 was not included in previous c…
821e58e 
…ommit
@cpyrgas
Copy link
Author

cpyrgas commented Jul 9, 2024

Confirmed fixed

@cpyrgas cpyrgas closed this as completed Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RobertvanderHulst RobertvanderHulst

Labels
bug Compiler
Projects
Build 2.x
Status: Done
Milestone
2.x
Development

No branches or pull requests
2 participants
@RobertvanderHulst @cpyrgas