search

SerenityOS / serenity

The Serenity Operating System 🐞
https://serenityos.org
BSD 2-Clause "Simplified" License
29.46k stars 3.14k forks source link

Kernel: Clean up duplicated code #24551

Closed brody-qq closed 3 weeks ago

brody-qq commented 3 weeks ago

There are 2 methods in InodeVMObject that are almost identical. This commit makes them both use the same code path.

BertalanD commented 3 weeks ago

nit: "cleanup" is a noun, "clean up" is the verb.

brody-qq commented 3 weeks ago

nit: "cleanup" is a noun, "clean up" is the verb.

ok, I changed the titles for the PR and commit :+1:

brody-qq commented 3 weeks ago

Thanks for the feedback

I changed the commit title and message to be a bit more specific