From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: "Pascal J. Bourguignon" Newsgroups: gmane.emacs.help Subject: Re: How to delete all nil properties from a plist? Date: Sun, 09 Aug 2015 04:18:16 +0200 Organization: Informatimago Message-ID: <87zj21gq6v.fsf@kuiper.lan.informatimago.com> References: <87oaiq3buh.fsf@mbork.pl> <87mvya3bij.fsf@mbork.pl> <871tfmv5d5.fsf@kuiper.lan.informatimago.com> <87si81u09e.fsf@kuiper.lan.informatimago.com> <87mvy5nwid.fsf@kuiper.lan.informatimago.com> <87egjhnnpq.fsf@kuiper.lan.informatimago.com> <87io8qjuw4.fsf@kuiper.lan.informatimago.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Trace: ger.gmane.org 1439086832 22341 80.91.229.3 (9 Aug 2015 02:20:32 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sun, 9 Aug 2015 02:20:32 +0000 (UTC) To: help-gnu-emacs@gnu.org Original-X-From: help-gnu-emacs-bounces+geh-help-gnu-emacs=m.gmane.org@gnu.org Sun Aug 09 04:20:20 2015 Return-path: Envelope-to: geh-help-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1ZOGDj-0004I9-Ti for geh-help-gnu-emacs@m.gmane.org; Sun, 09 Aug 2015 04:20:20 +0200 Original-Received: from localhost ([::1]:54202 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZOGDj-0005hl-5e for geh-help-gnu-emacs@m.gmane.org; Sat, 08 Aug 2015 22:20:19 -0400 Original-Path: usenet.stanford.edu!fu-berlin.de!uni-berlin.de!individual.net!not-for-mail Original-Newsgroups: gnu.emacs.help Original-Lines: 51 Original-X-Trace: individual.net nyNyfAimrDLfqfY5pt3kLAzDaNBIEf5SBxi70VTXxmVCs+/oYI Cancel-Lock: sha1:OWZlOWJiYjlkOTA2NzM3ODJiMDlhNGMyY2QzZDc1MzA3OTJlOTU5MQ== sha1:vLBhCtscG3FulYrwILZvVJ7Ic3E= Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwAQMAAABtzGvEAAAABlBMVEUAAAD///+l2Z/dAAAA oElEQVR4nK3OsRHCMAwF0O8YQufUNIQRGIAja9CxSA55AxZgFO4coMgYrEDDQZWPIlNAjwq9 033pbOBPtbXuB6PKNBn5gZkhGa86Z4x2wE67O+06WxGD/HCOGR0deY3f9Ijwwt7rNGNf6Oac l/GuZTF1wFGKiYYHKSFAkjIo1b6sCYS1sVmFhhhahKQssRjRT90ITWUk6vvK3RsPGs+M1RuR mV+hO/VvFAAAAABJRU5ErkJggg== X-Accept-Language: fr, es, en User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) Original-Xref: usenet.stanford.edu gnu.emacs.help:214088 X-BeenThere: help-gnu-emacs@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Users list for the GNU Emacs text editor List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: help-gnu-emacs-bounces+geh-help-gnu-emacs=m.gmane.org@gnu.org Original-Sender: help-gnu-emacs-bounces+geh-help-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.help:106373 Archived-At: Emanuel Berg writes: > "Pascal J. Bourguignon" > writes: > >>>> NEVER put the then on the same line as the test! >>> >>> ... why? >> >> because it makes it confusing to read. > > I think it looks good. Often the condition is short > (it should be short) so there is plenty of space to > use on that line. > >> If you do that, we have to count the parentheses!!! > > Why, and when? Compare those two forms: (if (and (some-predicate-p a) (= a 1) (print (list a (some-function a)))) (print b)) (if (and (some-predicate-p a) (= a 1)) (print (list a (some-function a))) (print b)) and then compare those two forms: (when (and (some-predicate-p a) (= a 1) (print (list a (some-function a)))) (print b)) (if (and (some-predicate-p a) (= a 1)) (print (list a (some-function a))) (print b)) > I have done that all the time and not once counted > parentheses: > > (if nil-prop (setq nil-prop nil) > (setq new (cons x new)) ) This is very bad style. -- __Pascal Bourguignon__ http://www.informatimago.com/ “The factory of the future will have only two employees, a man and a dog. The man will be there to feed the dog. The dog will be there to keep the man from touching the equipment.” -- Carl Bass CEO Autodesk