Paths

Table of Contentst

Differential D19221

Use iconv to ensure the output of calendar respects the output encoding
ClosedPublic
Actions

Authored by bapt on Feb 17 2019, 10:56 AM.

Tags

None

Referenced Files

	F136387838: D19221.id.diff
	Mon, Nov 17, 1:24 PM

	Unknown Object (File)
	Sun, Nov 16, 7:39 AM

	Unknown Object (File)
	Sat, Nov 15, 6:44 PM

	Unknown Object (File)
	Sat, Nov 15, 2:17 AM

	Unknown Object (File)
	Fri, Nov 14, 11:47 AM

	Unknown Object (File)
	Thu, Nov 13, 2:42 PM

	Unknown Object (File)
	Thu, Nov 13, 4:37 AM

	Unknown Object (File)
	Tue, Nov 11, 9:25 PM

Subscribers

Details

Reviewers

yuripv
jilles

Commits

rS345577: MFC r344340:
rS344340: calendar: use iconv to respect the output encoding

Summary

Use iconv to ensure the output of calendar respects the output encoding

Test Plan

tested with french and russian calendars

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bapt created this revision.Feb 17 2019, 10:56 AM

Herald added a subscriber: imp. · View Herald TranscriptFeb 17 2019, 10:56 AM

Harbormaster completed remote builds in B22538: Diff 54008.Feb 17 2019, 10:56 AM

bapt added a reviewer: yuripv.Feb 17 2019, 11:01 AM

jilles added a subscriber: jilles.Feb 17 2019, 1:03 PM

jilles added inline comments.

usr.bin/calendar/calendar.c
90–95 ↗	(On Diff #54008)	Perhaps `nl_langinfo(CODESET)` is better than hard-coding the locale name format.
usr.bin/calendar/events.c
83 ↗	(On Diff #54008)	Style: the open brace should be on a line by itself.
198 ↗	(On Diff #54008)	This strdup is unnecessary. The pointer can be saved instead and freed after the asprintf.

Address @jilles comments

Harbormaster completed remote builds in B22539: Diff 54011.Feb 17 2019, 3:41 PM

bapt added a reviewer: jilles.Feb 17 2019, 3:42 PM

Fix style

Harbormaster completed remote builds in B22540: Diff 54012.Feb 17 2019, 3:43 PM

bapt marked 2 inline comments as done.Feb 17 2019, 3:44 PM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 20 2019, 6:41 AM

Closed by commit rS344340: calendar: use iconv to respect the output encoding (authored by bapt). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.bin/

calendar/

4 lines

7 lines

10 lines

134 lines

3 lines

Diff 54124

head/usr.bin/calendar/Makefile

Loading...

head/usr.bin/calendar/calendar.h

Loading...

head/usr.bin/calendar/calendar.c

Loading...

head/usr.bin/calendar/events.c

Loading...

head/usr.bin/calendar/io.c

Loading...