Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
Message from discussion code review 6256048: exp/locale/collate: avoid 16-bit math (issue 6256048)

Received: by 10.180.98.167 with SMTP id ej7mr107426wib.1.1337885444079;
        Thu, 24 May 2012 11:50:44 -0700 (PDT)
X-BeenThere: golang-dev@googlegroups.com
Received: by 10.220.227.74 with SMTP id iz10ls764349vcb.8.gmail; Thu, 24 May
 2012 11:50:43 -0700 (PDT)
Received: by 10.52.29.193 with SMTP id m1mr1389398vdh.4.1337885443192;
        Thu, 24 May 2012 11:50:43 -0700 (PDT)
Received: by 10.52.29.193 with SMTP id m1mr1389396vdh.4.1337885443179;
        Thu, 24 May 2012 11:50:43 -0700 (PDT)
Return-Path: <3A4O-Tw0JCoYmyno1o5so6-r1qwksv.mywqyvkxq-no5qyyqvoq1y4z2....@m3kw2wvrgufz5godrsrytgd7.apphosting.bounces.google.com>
Received: from mail-vc0-f200.google.com (mail-vc0-f200.google.com [209.85.220.200])
        by gmr-mx.google.com with ESMTPS id l12si65658vdf.3.2012.05.24.11.50.43
        (version=TLSv1/SSLv3 cipher=OTHER);
        Thu, 24 May 2012 11:50:43 -0700 (PDT)
Received-SPF: pass (google.com: domain of 3A4O-Tw0JCoYmyno1o5so6-r1qwksv.mywqyvkxq-no5qyyqvoq1y4z2....@m3kw2wvrgufz5godrsrytgd7.apphosting.bounces.google.com designates 209.85.220.200 as permitted sender) client-ip=209.85.220.200;
Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of 3A4O-Tw0JCoYmyno1o5so6-r1qwksv.mywqyvkxq-no5qyyqvoq1y4z2....@m3kw2wvrgufz5godrsrytgd7.apphosting.bounces.google.com designates 209.85.220.200 as permitted sender) smtp.mail=3A4O-Tw0JCoYmyno1o5so6-r1qwksv.mywqyvkxq-no5qyyqvoq1y4z2....@m3kw2wvrgufz5godrsrytgd7.apphosting.bounces.google.com
Received: by vcqp1 with SMTP id p1so145502vcq.11
        for <golang-dev@googlegroups.com>; Thu, 24 May 2012 11:50:43 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.101.137.40 with SMTP id p40mr301243ann.12.1337885443028; Thu,
 24 May 2012 11:50:43 -0700 (PDT)
Reply-To: r...@golang.org, golang-dev@googlegroups.com, r...@golang.org, 
	re...@codereview-hr.appspotmail.com
Message-ID: <0016e68ee23e94f83704c0ccbaa5@google.com>
Date: Thu, 24 May 2012 18:50:43 +0000
Subject: Re: code review 6256048: exp/locale/collate: avoid 16-bit math (issue 6256048)
From: r...@golang.org
To: r...@golang.org
Cc: golang-dev@googlegroups.com, r...@golang.org, 
	re...@codereview-hr.appspotmail.com
Content-Type: text/plain; charset=ISO-8859-1; format=flowed; delsp=yes

*** Submitted as
http://code.google.com/p/go/source/detail?r=67d925cf4352 ***

exp/locale/collate: avoid 16-bit math

There's no need for the 16-bit arithmetic here,
and it tickles a long-standing compiler bug.
Fix the exp code not to use 16-bit math and
create an explicit test for the compiler bug.

R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/6256048


http://codereview.appspot.com/6256048/