> On Dec 5, 2018, at 11:23 AM, Segher Boessenkool <seg...@kernel.crashing.org>
> wrote:
>
> On Wed, Dec 05, 2018 at 02:19:14AM +0100, Stefan Kanthak wrote:
>> "Paul Koning" <paulkon...@comcast.net> wrote:
>>
>>> Yes, that's a rather nasty cut & paste error I made.
>>
>> I suspected that.
>> Replacing
>> !(den & (1L<<31))
>> with
>> (signed short) den >= 0
>> avoids this type of error: there's no need for a constant here!
>>
>> JFTR: of course the 1L should be just a 1, without suffix.
>
> "int" can be 16 bits only, I think? If you change to 15 you can remove
> the L, sure.
"int" on pdp11 is either 16 or 32 bits depending on switches, but "short int"
is always 16. I changed it to be 1U << 15 which seems more appropriate if int
is 16 bits.
paul
