X-Authentication-Warning: delorie.com: mail set sender to geda-user-bounces using -f
X-Recipient: geda-user AT delorie DOT com
Message-ID: <57CEEF82.4060208@xs4all.nl>
Date: Tue, 06 Sep 2016 18:32:02 +0200
From: "Bert Timmerman (bert DOT timmerman AT xs4all DOT nl) [via geda-user AT delorie DOT com]" <geda-user AT delorie DOT com>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.19) Gecko/20110429 Fedora/2.0.14-1.fc13 SeaMonkey/2.0.14
MIME-Version: 1.0
To: geda-user AT delorie DOT com
Subject: Re: [geda-user] pcb bugreport: ID type
References: <alpine DOT DEB DOT 2 DOT 00 DOT 1609060529360 DOT 7286 AT igor2priv>
In-Reply-To: <alpine.DEB.2.00.1609060529360.7286@igor2priv>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Reply-To: geda-user AT delorie DOT com

gedau AT igor2 DOT repo DOT hu wrote:
> Hi all,
>
> in the 20140316 release of mainline this bug is still present: 
> ANYOBJECTFIELDS has the ID field as long int, but create.c counts IDs 
> of new objects in an int.
>
> (This won't cause problems on the usual systems (with >=32 bit ints), 
> but the C standard permits int to be 16 bits wide in which case 
> create.c would start reusing IDs after 32767 objects. In the same time 
> there seems to be no drawback using long int in create.c)
>
> I propose using long int ID in create.c. Fixed in pcb-rnd in r2938.
>
> Regards,
>
> Igor2
>
>
>
Hello Igor2,

Thanks, fix is now committed in master.

Kind regards,

Bert Timmerman.