Paths

Table of Contentst

fix strchr, strrchr implementation on MIPS
ClosedPublic
Actions

Authored by br on Oct 13 2016, 11:40 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jan 31 2024, 10:20 AM

	Unknown Object (File)
	Dec 27 2023, 5:33 PM

	Unknown Object (File)
	Dec 27 2023, 5:08 PM

	Unknown Object (File)
	Dec 18 2023, 3:20 AM

	Unknown Object (File)
	Dec 10 2023, 7:06 PM

	Unknown Object (File)
	Dec 5 2023, 3:03 AM

	Unknown Object (File)
	Sep 28 2023, 9:02 PM

	Unknown Object (File)
	Sep 26 2023, 6:34 PM

View All 28 Files

Subscribers

None

Details

Reviewers

emaste
andrew

Commits

rS307219: Fix strchr, strrchr implementation: convert c to char

Summary

compare byte with byte, not byte with int

this is a fix for the following tests:

bin/sh/expansion/functional_test:ifs3
bin/sh/expansion/functional_test:plus_minus6
lib/libc/string/strchr_test:strchr_basic
lib/libc/string/strrchr_test:strrchr_basic

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

br updated this revision to Diff 21350.Oct 13 2016, 11:40 AM

br retitled this revision from to fix strchr, strrchr implementation on MIPS.

br updated this object.

br edited the test plan for this revision. (Show Details)

br added reviewers: emaste, andrew.

This looks correct, http://pubs.opengroup.org/onlinepubs/9699919799/functions/strchr.html states c should be converted to a char.

LGTM

This revision is now accepted and ready to land.Oct 13 2016, 12:06 PM

Closed by commit rS307219: Fix strchr, strrchr implementation: convert c to char (authored by br). · Explain WhyOct 13 2016, 3:24 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

lib/

libc/

mips/

string/

strchr.S

5 lines

strrchr.S

5 lines

Diff 21353

View Options

head/lib/libc/mips/string/strchr.S