From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: martin rudalics Newsgroups: gmane.emacs.devel Subject: Re: window-text-pixel-size and presumable move_it_to change(s) Date: Fri, 24 Jun 2016 10:09:34 +0200 Message-ID: <576CEABE.9010706@gmx.at> References: <576B9B4C.5080503@gmx.at> <83twgk2c1i.fsf@gnu.org> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Trace: ger.gmane.org 1466756875 8104 80.91.229.3 (24 Jun 2016 08:27:55 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Fri, 24 Jun 2016 08:27:55 +0000 (UTC) Cc: emacs-devel@gnu.org To: Eli Zaretskii Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Fri Jun 24 10:27:46 2016 Return-path: Envelope-to: ged-emacs-devel@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 1bGMSj-00080F-QJ for ged-emacs-devel@m.gmane.org; Fri, 24 Jun 2016 10:27:41 +0200 Original-Received: from localhost ([::1]:41845 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGMSd-0002Ua-SX for ged-emacs-devel@m.gmane.org; Fri, 24 Jun 2016 04:27:35 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:60016) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGMBL-0008DQ-Fj for emacs-devel@gnu.org; Fri, 24 Jun 2016 04:09:44 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bGMBK-0000CK-Iy for emacs-devel@gnu.org; Fri, 24 Jun 2016 04:09:43 -0400 Original-Received: from mout.gmx.net ([212.227.15.15]:51175) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bGMBG-00009M-GA; Fri, 24 Jun 2016 04:09:38 -0400 Original-Received: from [192.168.1.100] ([212.95.7.119]) by mail.gmx.com (mrgmx002) with ESMTPSA (Nemesis) id 0LgIWi-1bju7Q0vv2-00nktd; Fri, 24 Jun 2016 10:09:36 +0200 In-Reply-To: <83twgk2c1i.fsf@gnu.org> X-Provags-ID: V03:K0:dkE5XRnvURpiGdKhURpkGFLndv3et4YpvktM8e+cpfJZJHEYVGG /vSdxqgfyzGg+z3lkQFuGSRHraAe25iwQTeUZ0oeDjdQNfUFglT4AfSauixym2cx8Na0hkL 4x3jDG40KKk5jVOejgOaVHMLeebWGPwwk/05PABVa/HnRz9jmFuuSDZSUTNCgKJAbgr0wEu V3kSZ97O91uxsv7e6/cKg== X-UI-Out-Filterresults: notjunk:1;V01:K0:W89DeXN9SVU=:xkB5TxGLKTYYToCwvVsW8M cLC/7jxSZ0T3p7rWivH6yH6bM0LYIRGcpk65y10nReaFdN7Z6P+Es5Zr5oe7GxcTcntwp/Bvh od+kirXGZox9r5Kgy6DQao9FC7rEFw70NP8mPqTz8oPIfbnoFV/g40Xnz0YS3YunE9s1baePC 489oQKKdcgjmNS5z9mUuhoPwtrgmJ5ywgUarZJvJXJOmIqXkWUzWMDM9OPX/QGtSgu9I2LQQj A5kxKetCd+GaFkcPrAIlJOeflEZsB51OhKoidZeAhZZbOtwuQiZmaYwtBsSO/qlKYAoPZskeu 0eCQraLp+1hLNG5z6iiYkDCewxb1CJbic9HgXCMl+lHQveuPJ/eaHjq/KZihYHt+z0/K21Ao+ 0CzYBNE4wjRw2MMMjjZMkwN9DpKQWU8Z34djIvMQJeQiR1nsm+OrCcdqH1C/OYE8vdJOD4rwn oVy/Ctpv4k9CyekQr+XpUjyswFxpqvMbFHfgm7p3r2wmGujV9CaqQoaQQEO7xYY1852G5qT1e OpO2441VrZaofayAtbHaAYSQMvPqwqkuk/rmKyYktAvEqJ2lIqyiwPYvqgzlOIYbsyFvttwc7 WS14bBS4Fvu47q4u3P5arb2+damVn5pw6OVJe5SZoAtwyTo5xA6ugMn35Zq29ON8fGNE1x31X IRwKHiOx9Ed3c1ps1cDCl/smzSL/vOvzRQwK8YL9i1SBtFOPzfnkDKG9b8ssKLOL+5wgBDbNX 0gZIDMxKEdnXxhxseru2/1lrGO6aGEx9gx9o5Mk/gKnKh1kJfbGQwlHcW6yyrcTaG05lyVA1 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 212.227.15.15 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:204719 Archived-At: > The problem is in the new code > in Fwindow_text_pixel_size: > > x = min (move_it_to (&it, end, INT_MAX, max_y, -1, > MOVE_TO_POS | MOVE_TO_X | MOVE_TO_Y), > max_x); > > This will call move_it_to twice, and the second call will return a > different value from the correct one, returned by the first call. > Replace this by something less fancy, like > > x = move_it_to (&it, end, INT_MAX, max_y, -1, > MOVE_TO_POS | MOVE_TO_X | MOVE_TO_Y); > if (x > max_x) > x = max_x; > > and Bob'll be your uncle. That was the cause. Thanks for the solution. > (In general, I suggest to make a rule to never use 'min' or 'max' when > their arguments are expressions.) Indeed, I have to be more careful in this regard. Many thanks, martin