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

fix - fix cp_typo #1033

Closed
wants to merge 2 commits into from
Closed

Conversation

chiajungchang2003
Copy link

@chiajungchang2003 chiajungchang2003 commented Jan 12, 2024

  • line 343 for correct path -> for the correct path concatenation
  • line 430 optimise -> optimize
    Event: Advanced UNIX programming course (Fall'23) at NTHU

- line 343 for correct path -> for the correct path concatenation
- line 430 optimise -> optimize
- Event: Advanced UNIX programming course (Fall'23) at NTHU
@lwhsu
Copy link
Member

lwhsu commented Feb 13, 2024

Optimise is fine as it's British spelling.

freebsd-git pushed a commit that referenced this pull request Feb 13, 2024
Event:		Advanced UNIX programming course (Fall'23) at NTHU
Pull Request:	#1033
@lwhsu
Copy link
Member

lwhsu commented Feb 13, 2024

Merged as 416fdc2

@lwhsu lwhsu closed this Feb 13, 2024
@lwhsu lwhsu added the merged label Feb 13, 2024
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#1033
@dag-erling
Copy link
Member

dag-erling commented Mar 28, 2024

A bit late but “for correct path concatenation” was also correct. If you want to add an article you'll also have to invert the (nouned) verb and object, e.g. “for the correct concatenation of paths”, and even then the article is optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants