From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: util-linux-owner@vger.kernel.org Received: from mail-wm0-f66.google.com ([74.125.82.66]:35704 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752969AbdBCWle (ORCPT ); Fri, 3 Feb 2017 17:41:34 -0500 Received: by mail-wm0-f66.google.com with SMTP id u63so7512259wmu.2 for ; Fri, 03 Feb 2017 14:41:34 -0800 (PST) Date: Fri, 3 Feb 2017 22:41:18 +0000 (GMT) From: Sami Kerola To: J William Piggott cc: util-linux@vger.kernel.org, Karel Zak Subject: Re: [ping] Karel: Re: pull: hwclock 27 changes In-Reply-To: <9798a869-fb09-77e7-a13a-bca002617672@gmx.com> Message-ID: References: <20161231214107.8658-1-kerolasa@iki.fi> <05e4405f-8096-9261-f9f2-d2c6b84675bc@gmx.com> <070fc458-ed69-a09c-11cc-42bb50fec888@gmx.com> <5feac9a5-e3e3-5148-60a0-fb3197a1caa8@gmx.com> <9798a869-fb09-77e7-a13a-bca002617672@gmx.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: util-linux-owner@vger.kernel.org List-ID: On Thu, 2 Feb 2017, J William Piggott wrote: > Karel, > I have some work to submit that applies on top of Sami's branch: > > git://github.com/kerolasa/lelux-utiliteetit.git hwclock-jwp-reviewed Nice to hear. > Outside of our ongoing discussion below regarding a 'technically > unnecessary' cast, I think Sami's work is ready for you to consider it > for committing. I would be interested in your thoughts on the cast. I had a Friday conversation with my work colleagues and majority thought that 'unnecessary cast' is good thing, as explicit a cast is visual reminder of type change. We did consider maintenance burden of these casts, but counter argument was found to be difficult to object. It is true that unnecessary casts cause unnecessary change when either function return value or storage unit changes - but such changes do not happen often. Therefore weight of visual reminder is heavier than an attempt to keep potential future diffs minimal. With one thing everyone agreed, these sorts of little things should not get in way too much. In short cast or don't but do not get stuck with a detail like this. To be honest this is probably the best advice. While I would see these sorts of type change reminders I'm happy to rip them out if that is needed to move forward. p.s. Branch hwclock-jwp-reviewed in my git is rebased on top of most recent origin/master. -- Sami Kerola http://www.iki.fi/kerolasa/