having noSmooth(); in setup will trigger a duplicate settings() warning

[processing-bot]

Created by: clankill3r

Processing 4.3
Windows 10

I thought I was going crazy :)

void settings() {
 
}

void setup() {
  noSmooth();
}

[processing-bot]

Created by: JuzerNejm10

Yeah, happened to me a couple of times with noSmooth() or smooth(). That must go into settings() .
But, on the other hand, settings() are very restrictive, you cannot have eg. frameRate(), background(), stroke(), fill(), noCursor(), surface.setVisible() etc. in there - you'll get NullPointerException.

[Stefterv]

Seeing the same thing @clankill3r, I will look into it

benfry/processing4#823

[nightshining]

Same issue. Mac 4.3.3 Need to use fullscreen() in settings()

Duplicate method settings() in type

[wacala]

Same here. Processing 4.3.4 for Mac, M1 processor, macOS Sequuoia.

[SableRaf]

Thanks very much to @nightshining, @wacala, and everyone who reported this issue 💙

This is confirmed to be a bug, and we’re currently investigating. At this point, additional reports aren’t needed, but feel free to add a 👍 on the original post if you’re also encountering this issue. We’ll keep this thread updated as things progress.

Note

If you’re curious about the cause of this bug or want to help with a fix, the Processing code is open source and contributions are always welcome! Check out our contribution guidelines for instructions. We're also here to offer guidance to new contributors so if you need help getting started, feel free to ask.

[SableRaf]

@Gargaj said in #861 (comment)

I looked into it briefly (just by reading the repo) and I suspect it has something to do with https://github.com/processing/processing4/blob/main/java/src/processing/mode/java/preproc/PdeParseTreeListener.java#L372-L380