In D38496#877177, @des wrote:

In D38496#876169, @lwhsu wrote:

I also have some WIP tests for wc(1) to be added in D35877

It's unfortunate that this wasn't approved back in the day. Does it cover anything that my tests don't?

It's my fault that haven't completed it yet but I do plan to work on it with the submitter this week. Do you mind if I go ahead to remove the duplicated ones in D35877 and merge it first? Or you want to get this differential in first? I proposed to add tests first because it would be a good practice for code refactoring, but I can wait if you want.

Add descriptions to test cases.

Harbormaster completed remote builds in B49739: Diff 117104.Feb 13 2023, 5:13 PM

In D38496#877219, @lwhsu wrote:

In D38496#877177, @des wrote:

In D38496#876169, @lwhsu wrote:

I also have some WIP tests for wc(1) to be added in D35877

It's unfortunate that this wasn't approved back in the day. Does it cover anything that my tests don't?

It's my fault that haven't completed it yet but I do plan to work on it with the submitter this week. Do you mind if I go ahead to remove the duplicated ones in D35877 and merge it first? Or you want to get this differential in first? I proposed to add tests first because it would be a good practice for code refactoring, but I can wait if you want.

How about we add a comment to wc_test.sh crediting the student?

des edited the summary of this revision. (Show Details)Feb 13 2023, 8:03 PM

kevans accepted this revision as: Klara, kevans.Feb 13 2023, 8:50 PM

This revision is now accepted and ready to land.Feb 13 2023, 8:50 PM

Add a few more test and fix the whitespace test.

This revision now requires review to proceed.Feb 14 2023, 11:47 AM

Harbormaster completed remote builds in B49772: Diff 117194.Feb 14 2023, 11:47 AM

Nit: alphabetize envars.

Harbormaster completed remote builds in B49778: Diff 117202.Feb 14 2023, 2:39 PM

kevans accepted this revision as: Klara, kevans.Feb 14 2023, 7:18 PM

This revision is now accepted and ready to land.Feb 14 2023, 7:18 PM

Closed by commit rGdad64f0e7cad: wc: Clean up and modernize.. · Explain WhyFeb 14 2023, 10:27 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGdad64f0e7cad: wc: Clean up and modernize..

wc: Clean up and modernize.
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 116960

etc/mtree/BSD.tests.dist

usr.bin/wc/Makefile

usr.bin/wc/foo

usr.bin/wc/tests/Makefile

usr.bin/wc/tests/wc_test.sh

usr.bin/wc/wc.c

wc: Clean up and modernize.ClosedPublicActions