poke-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 1/2] poke: fix handling of alien tokens


From: Mohammad-Reza Nabipoor
Subject: Re: [PATCH v2 1/2] poke: fix handling of alien tokens
Date: Wed, 1 Nov 2023 14:19:44 +0100

On Wed, Nov 01, 2023 at 02:10:54PM +0100, Jose E. Marchesi wrote:
> 
> > +++ b/poke/pk-ios.c
> > @@ -245,10 +245,10 @@ pk_ios_alien_token_handler (char delimiter,
> >  
> >    assert (delimiter == '<');
> >  
> > -  if (id_len == 3)
> > +  if (id_len == 2)  /* <> */
> >      goto error;
> 
> I think that was supposed to be < 2.  Empty $<> can be weird, but useful
> at some point.
> 

Then, if there's no IOS:
    $<> leads to "invalid IO space"
if there's on IOS:
    $<> represent it.
else:
    $<> leads to "invalid IO space"


For me this sounds good.
Do you have any objections?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]