Bug 23866

Summary:	patch for pointing out current date
Product:	Base System	Reporter:	aaron <aaron>
Component:	bin	Assignee:	dwmalone
Status:	Closed FIXED
Severity:	Affects Only Me
Priority:	Normal
Version:	1.0-RELEASE
Hardware:	Any
OS:	Any

Description aaron 2000-12-26 23:20:01 UTC

	this patch adds the option -h, which highlights the current
	date in a calendar listing by parenthesizing it.

	the patch is two diffs - .c and .1. since this program doesn't
	appear to support attaching files, i have posted a 1K tarball
	of my changes at: http://lithic.org/aaron/ncal-diffs.tar.gz

	please contact me by email if there are any problems or questions
	about my patch. thanks!

Comment 1 dwmalone 2000-12-26 23:52:41 UTC

On Tue, Dec 26, 2000 at 03:14:20PM -0800, aaron@eazel.com wrote:

> 	the patch is two diffs - .c and .1. since this program doesn't
> 	appear to support attaching files, i have posted a 1K tarball
> 	of my changes at: http://lithic.org/aaron/ncal-diffs.tar.gz

I'm having a look at your patch - the idea seems nice, but the
code has some serious style and whitespace issues. Have you
done "man 9 style" to read about the style usually used for
FreeBSD code? Also, we usually use unified diffs (diff -u)
as they are easier to read through.

I'll let you know when I've a better look.

	David.

Comment 2 dwmalone freebsd_committer

2000-12-26 23:53:15 UTC

Responsible Changed
From-To: freebsd-bugs->dwmalone

I'm having a look at the patch.

Comment 3 dwmalone freebsd_committer

2005-02-08 09:30:21 UTC

State Changed
From-To: open->closed

I never managed to take any action on this PR (there were some 
style issues and layout of the output problems). The patch is 
no longer available either, so close this PR. 

David.