Paths

Table of Contentst

Differential D36833

rs: Convert to C++ to convert elem to a std::vector<char *>.
ClosedPublic
Actions

Authored by jhb on Sep 29 2022, 10:48 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Dec 1 2024, 6:07 PM

	Unknown Object (File)
	Nov 29 2024, 1:27 PM

	Unknown Object (File)
	Nov 22 2024, 10:45 PM

	Unknown Object (File)
	Nov 22 2024, 11:58 AM

	Unknown Object (File)
	Nov 22 2024, 9:39 AM

	Unknown Object (File)
	Nov 22 2024, 5:10 AM

	Unknown Object (File)
	Nov 19 2024, 10:55 PM

	Unknown Object (File)
	Nov 19 2024, 1:55 AM

View All 58 Files

Subscribers

Details

Reviewers

emaste
markj
brooks

Commits

rG42d10b1b56f2: rs: Convert to C++ to convert elem to a std::vector<char *>.

Summary

This also updates various indices and counters from int to size_t to
pacify resulting -Wsign-compare warnings.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 47647
Build 44534: arc lint + arc unit

Event Timeline

jhb created this revision.Sep 29 2022, 10:48 PM

Herald added a subscriber: imp. · View Herald TranscriptSep 29 2022, 10:48 PM

jhb requested review of this revision.Sep 29 2022, 10:48 PM

Harbormaster completed remote builds in B47647: Diff 111253.Sep 29 2022, 10:48 PM

jhb added a parent revision: D36832: rs: Fix some pointer arith UB..Sep 29 2022, 10:48 PM

jhb added a child revision: D36834: rs: Use getopt() and strtol() instead of mannual parsing..

jhb added inline comments.Sep 29 2022, 10:57 PM

usr.bin/rs/rs.cc
186	Nothing seemed to depend on this NULL being present, and `nelem` doesn't include it.
323	This seems like dead code. At line 280 above in the old code we set `nelem` to `n` above. The only way we wouldn't execute that line was if `lp` already fit which would imply that `nelem` was greater than `orows * ocols` on function entry. Either way, `nelem` would never be larger than `n` if the RECYCLE flag was set.

This might need a fixup depends rule due to rs.c being renamed to rs.cc.

brooks accepted this revision.Sep 29 2022, 11:20 PM

This revision is now accepted and ready to land.Sep 29 2022, 11:20 PM

Closed by commit rG42d10b1b56f2: rs: Convert to C++ to convert elem to a std::vector<char *>. (authored by jhb). · Explain WhyNov 16 2022, 5:19 AM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG42d10b1b56f2: rs: Convert to C++ to convert elem to a std::vector<char *>..

Revision Contents
Changeset List

Path

Size

usr.bin/

rs/

2 lines

	rs.cc
	rs.c

155 lines

Diff 111253

usr.bin/rs/Makefile

Loading...

usr.bin/rs/rs.c

Loading...

usr.bin/rs/rs.cc

Loading...