Skip to content

Commented out debug statement causing compilation failure on OSX.#35

Merged
daschuer merged 2 commits into
mixxxdj:masterfrom
cpill0789:compilation_fix
Jul 6, 2013
Merged

Commented out debug statement causing compilation failure on OSX.#35
daschuer merged 2 commits into
mixxxdj:masterfrom
cpill0789:compilation_fix

Conversation

@cpill0789
Copy link
Copy Markdown
Contributor

Two debugging lines introduced in dlgprefsound.cpp in commit aa2f6bf caused compilation failure on Mac OSX because they referenced Linux specific rlimit code. Removing them fixes the compilation failure.

Comment thread src/dlgprefsound.cpp Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe wrap these in a linux-specific ifdef instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That makes sense. I can enclose them in the ifdef that follows, which weirdly doesn't encapsulate the body of the if statement. I compared it to older code and the if body code is new and should be included in the ifdef as far as I can tell.

@daschuer
Copy link
Copy Markdown
Member

daschuer commented Jul 6, 2013

Maybe we can drop these debug messages entirely later. But for now it is a good solution. Thank you Carl!

daschuer added a commit that referenced this pull request Jul 6, 2013
fix compilation failure on OSX.
@daschuer daschuer merged commit 09c731a into mixxxdj:master Jul 6, 2013
@cpill0789 cpill0789 deleted the compilation_fix branch July 9, 2013 02:21
@daschuer daschuer mentioned this pull request Dec 28, 2014
@daschuer daschuer mentioned this pull request Jun 11, 2018
11 tasks
Holzhaus pushed a commit to Holzhaus/mixxx that referenced this pull request Aug 23, 2020
Holzhaus pushed a commit to Holzhaus/mixxx that referenced this pull request Aug 23, 2020
m0dB pushed a commit to m0dB/mixxx that referenced this pull request Jan 21, 2024
koushikcs562 pushed a commit to koushikcs562/mixxx that referenced this pull request Dec 23, 2025
Signed-off-by: Dscano <d.scano89@gmail.com>
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.

3 participants