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

share/examples/sound/midi.c, share/examples/sound/ossmidi.h typo #994

Closed
wants to merge 1 commit into from

Conversation

Assume-Zhan
Copy link
Contributor

Typo in line 52 of share/examples/sound/midi.c: CONTROLER_ON -> CONTROLLER_ON
Typo in line 68 of share/examples/sound/midi.c: CONTROLER_ON -> CONTROLLER_ON
Typo in line 36 of share/examples/sound/ossmidi.h: CONTROLER_ON -> CONTROLLER_ON

Event: This is from the Advanced UNIX Programming Course (Fall’23) at NTHU.

@lwhsu
Copy link
Member

lwhsu commented Jan 5, 2024

The name and email you provided in the comment are different from what's in https://patch-diff.githubusercontent.com/raw/freebsd/freebsd-src/pull/994.patch , which is your preference?

@Assume-Zhan
Copy link
Contributor Author

Sorry about that, I prefer use the name and email below:

@lwhsu
Copy link
Member

lwhsu commented Jan 6, 2024

Ok, then please amend your commit (git commit --amend) and do a force push to update the commit in this PR. While doing so, please also update the commit message to a more descriptive one, in the subject and body, and with

Event: Advanced UNIX Programming Course (Fall’23) at NTHU.

In the bottom of the commit message.

In line 52 of share/examples/sound/midi.c: CONTROLER_ON should be CONTROLLER_ON
In line 68 of share/examples/sound/midi.c: CONTROLER_ON should be CONTROLLER_ON
In line 36 of share/examples/sound/ossmidi.h: CONTROLER_ON should be CONTROLLER_ON

Event: Advanced UNIX Programming Course (Fall’23) at NTHU.
@lwhsu
Copy link
Member

lwhsu commented Feb 13, 2024

Merged as 0789c3b

@lwhsu lwhsu closed this Feb 13, 2024
@lwhsu lwhsu added the merged label Feb 13, 2024
freebsd-git pushed a commit that referenced this pull request Feb 13, 2024
Event:		Advanced UNIX Programming Course (Fall’23) at NTHU.
Pull Request:	#994
5u623l20 pushed a commit to 5u623l20/freebsd-src that referenced this pull request Feb 16, 2024
Event:		Advanced UNIX Programming Course (Fall’23) at NTHU.
Pull Request:	freebsd#994
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants