FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

DEBUG should be set to the actual value - Fix for issue-1245 by SuganthiThomas · Pull Request #1247 · processing/processing4 · GitHub

DEBUG should be set to the actual value - Fix for issue-1245 - #1247

Merged
catilac merged 2 commits into
processing:mainfrom
SuganthiThomas:fix-issue-1245
Sep 18, 2025
Merged

DEBUG should be set to the actual value - Fix for issue-1245#1247
catilac merged 2 commits into
processing:mainfrom
SuganthiThomas:fix-issue-1245

Conversation

Copy link
Copy Markdown
Contributor

Hi,

I have made changes in file processing4\app\src\processing\app\Base.java (Package: processing.app)

in line 69

static public boolean DEBUG = Boolean.parseBoolean(System.getenv().getOrDefault("DEBUG", "false"));

Please review it.

Thank You

catilac left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Yay this works! Thank you so much. Please remove the commented out old code, and then I'll approve.

catilac commented Sep 17, 2025

Copy link
Copy Markdown
Collaborator

Fixes #1245

SuganthiThomas commented Sep 18, 2025 via email

Copy link
Copy Markdown
Contributor Author

catilac left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thank you @SuganthiThomas !

catilac merged commit 9ca92bc into processing:main Sep 18, 2025
6 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL