From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 110748 invoked by alias); 14 Jun 2017 00:56:27 -0000 Mailing-List: contact kawa-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: kawa-owner@sourceware.org Received: (qmail 110725 invoked by uid 89); 14 Jun 2017 00:56:26 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: aibo.runbox.com Received: from aibo.runbox.com (HELO aibo.runbox.com) (91.220.196.211) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 14 Jun 2017 00:56:23 +0000 Received: from [10.9.9.211] (helo=mailfront11.runbox.com) by mailtransmit02.runbox with esmtp (Exim 4.86_2) (envelope-from ) id 1dKwbg-0003ik-TR; Wed, 14 Jun 2017 02:56:24 +0200 Received: from 70-36-239-209.dsl.dynamic.fusionbroadband.com ([70.36.239.209] helo=localhost.localdomain) by mailfront11.runbox.com with esmtpsa (uid:757155 ) (TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.82) id 1dKwbS-0001VZ-Fl; Wed, 14 Jun 2017 02:56:10 +0200 Subject: Re: Exception on pprint-logical-block without 'out:' (patch incl.) To: Sudarshan S Chawathe , kawa References: <11182.1497220626@vereq.eip10.org> From: Per Bothner Message-ID: Date: Wed, 14 Jun 2017 00:56:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 MIME-Version: 1.0 In-Reply-To: <11182.1497220626@vereq.eip10.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2017-q2/txt/msg00109.txt.bz2 On 06/11/2017 03:37 PM, Sudarshan S Chawathe wrote: > > Evaluating a pprint-logical-block form without the optional 'out' > keyword argument throws an exception. (See details below.) It works > fine when that is provided. > > Based on a quick look at kawa/lib/kawa/pprint.scm, I suspect the > problem is just a missing (define out-expr #!null) in the > define-syntax-case. The attached trivial patch seems to do the trick, > but I have not tested carefully. (I can also submit using gitlab if > you prefer.) I checked this in. Thanks! -- --Per Bothner per@bothner.com http://per.bothner.com/