X-Authentication-Warning: delorie.com: mail set sender to djgpp-bounces using -f
X-Recipient: djgpp AT delorie DOT com
X-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=20161025;
        h=mime-version:in-reply-to:references:from:date:message-id:subject:to;
        bh=Z6+e02EdwLf/boH6Qkxthlffg4l36pmGXklt06YARdo=;
        b=mWMhE3/yafLF4B52+btten7m3fm+pVfrsuB4sEhgxCJvwXRecLaNaOzWhCZZs/hVxl
         Jr3wCFSp5/hDDJNszPDPpiXG+LdYtUtehE6xP2EQpN+8WkR4jH40cdXKKIj2xoSJthVI
         gP7DTZerdZNZMHE96JrTDBd9yWza82/vEfpM1aXhiSu+9iCN2G4OcfI1AkqErxPQIoB4
         TivL8Cza5tJQtn7TPDycm/qatx6ul54VXxNgBHzHZxFGHsQmNJtNQ492LMvhzMTpORXu
         Wj0fzmTKrbjoe0r5YhLZKY/gGwnTHwjlucV7gBoVz6w4cBxIb3CyV9xRarGzT4bYElvt
         fGnA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20161025;
        h=x-gm-message-state:mime-version:in-reply-to:references:from:date
         :message-id:subject:to;
        bh=Z6+e02EdwLf/boH6Qkxthlffg4l36pmGXklt06YARdo=;
        b=p9C3FSU7cAlIglqCdU7GC6DdCu2tL5aXUPza2ru0KOqHI2wbIhkC7zSx4e/pZ5EVhN
         OvPE2FQWLmQ4sZNgTnU8GdDz/nQEnAQTPCYGwf2VjZctIm6NhQCQffiOa6+xJN4qpDyk
         QVqlKgJI0vA9IA8TNVNVTUBWRnWrgQ0Gzlf+0vVBXU1aUB+Ueboyc//IhRLb8yER3oyn
         9wK7pccokGvqOtAuep22u4Jy3j1jEVLdItRUi5Ap0pTX2HDbdH1bhpwomxAIcGn1k+9X
         iDFp/uUtk84ciPt7tUYcaqAviTK5anRufQKq7eOX1q3R5SC/Xfroz/b4VvvkX1IZtnpQ
         AdPQ==
X-Gm-Message-State: AN3rC/4qq28xwB2SyQLQJazX+66HZCNXsPLoX+XTNPCKuDN8OK36QR+d
	6fHgZtDQNeVsz/4v4rj0h3K7mGhHUw==
X-Received: by 10.55.74.214 with SMTP id x205mr9244989qka.231.1493410802816;
 Fri, 28 Apr 2017 13:20:02 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <83d1bwmitb.fsf@gnu.org>
References: <CAA2C=vC8s9Kg9LhewQmUMtiJNm+f=Gu5b-m_QXgiTgNufQQpfQ AT mail DOT gmail DOT com>
 <CAA2C=vBNvQsBU+2_0JhJQqJr41pV=VWTUxgyS29VjUK6zL3paA AT mail DOT gmail DOT com>
 <CAA2C=vDKwsv7i5jNOGEpOEdokwsWbP3MV_eHU_M6s79cn=NkZg AT mail DOT gmail DOT com>
 <CAA2C=vC3EtQJmndPY7NRN_cSnDLbNpJPPM2FV6H4E9DcLoWgTg AT mail DOT gmail DOT com> <83d1bwmitb DOT fsf AT gnu DOT org>
From: "Ozkan Sezer (sezeroz AT gmail DOT com) [via djgpp AT delorie DOT com]" <djgpp AT delorie DOT com>
Date: Fri, 28 Apr 2017 23:20:02 +0300
Message-ID: <CAA2C=vByAyq=X2Xkj+wVQn+ZMZGxWsieuemO4eL-UGh9JSxoGQ@mail.gmail.com>
Subject: Re: dxe3gen does not handle relocs > 65535
To: djgpp AT delorie DOT com
Content-Type: text/plain; charset=UTF-8
Reply-To: djgpp AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

On 4/28/17, Eli Zaretskii (eliz AT gnu DOT org) [via djgpp AT delorie DOT com]
<djgpp AT delorie DOT com> wrote:
>> From: "Ozkan Sezer (sezeroz AT gmail DOT com) [via djgpp AT delorie DOT com]"
>> <djgpp AT delorie DOT com>
>> Date: Fri, 28 Apr 2017 22:20:39 +0300
>>
>> Here is a somewhat cleaned-up patch with a changelog entry:
>
> Thanks.  Some minor comments:
>
>> +    if (opt.verbose)
>> +      printf("%s: real nrelocs: %lu\n", progname, (unsigned
>> long)real_nrelocs);
>
> is this printf really necessary?

Not truly. I can comment it out.

>
>> +  } else {
>> +    real_nrelocs = dh.nrelocs;
>> +  }
>
> Please use the style adopted in the rest of the source, with regards
> to braces.

OK.

>
>> Should apply to CVS?
>
> If no one speaks up in a couple of days, please do.
>

OK.  Thanks.

--
O.S.