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.

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

Problem with function size(int arg, int arg) in Class #317

Closed
knupel opened this issue Dec 16, 2021 · 6 comments · Fixed by #384
Closed

Problem with function size(int arg, int arg) in Class #317

knupel opened this issue Dec 16, 2021 · 6 comments · Fixed by #384
Labels
Help Wanted We have very little time and would like some help Preprocessor

Comments

@knupel
Copy link
Contributor

knupel commented Dec 16, 2021

When the function void size(int arg, int arg) is used with two arguments in Class that's cause a syntax problem when this function is call directly in void setup()

Processing 4.0.2b / OS Monterey

Code to reproduce:

Truc truc = new Truc();
void setup() {
  size(200,200);
  truc.size(1,1); // problem >>> error à "."
  // func();
}

void draw() {
  truc.size(1,1); // no problem
}

void func() {
  truc.size(1,1); // no problem
}

class Truc {
  void size(int x, int y) {
  }
}
@knupel knupel changed the title Problem with function size(int arg, int arg); Problem with function size(int arg, int arg) in Class Dec 16, 2021
@benfry benfry added Preprocessor Help Wanted We have very little time and would like some help labels Jan 10, 2022
@ritikranjan12
Copy link

Hello, @StanLepunK are you still working on this. I have the answer of this problem,Can i make a Pull Request for the same?

@knupel
Copy link
Contributor Author

knupel commented Jan 12, 2022

@ritikranjan12 sure make a pull request for this problem, because is not my problem, it's a problem for all people on a same configuration of code. So if your fix can help all people GOGOGO make a pull request !

@ritikranjan12
Copy link

@StanLepunK I am a beginner can you help me to locate this code sample in the project?I can't found it.....or I just make a simple documentation for the above problem?

@knupel
Copy link
Contributor Author

knupel commented Jan 12, 2022

@ritikranjan12 Are youbeginner in Processing or in Java. For you're fix are you build a new Processing from branch with your fix from Processing 4 core ? See https://github.com/processing/processing4/blob/master/build/README.md
So if you've do that, that's great. After that you can create a pull request from your work branch.
For the other side I don't know where is exactly the bug to fix, because me too I'm not an expert @benfry add the bug in the Preprocessor... i know just that.

@tmatinla
Copy link

Cannot reproduce this with 4.0 beta 3 (it does happen with beta 2)

sampottinger added a commit that referenced this issue Jan 29, 2022
Though this was already fixed, just adding a test to confirm that it stays fixed. Need to respect calls to user-definined size methods including those defined in user-made classes.
@github-actions
Copy link

github-actions bot commented Mar 2, 2022

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Help Wanted We have very little time and would like some help Preprocessor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants