On Wed, May 16, 2018 at 12:24:06PM -0400, Frediano Ziglio wrote:
> >
> > On Wed, May 16, 2018 at 11:04:17AM -0400, Frediano Ziglio wrote:
> > > >
> > > > On Mon, May 14, 2018 at 11:18:53PM +0100, Frediano Ziglio wrote:
> > > > > Make sure there are not 2 messages with the same name in the
> > > > > same channel.
> > > > >
> > > > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> > > > > ---
> > > > > python_modules/ptypes.py | 3 +++
> > > > > 1 file changed, 3 insertions(+)
> > > > >
> > > > > diff --git a/python_modules/ptypes.py b/python_modules/ptypes.py
> > > > > index ef580aa..664dd1d 100644
> > > > > --- a/python_modules/ptypes.py
> > > > > +++ b/python_modules/ptypes.py
> > > > > @@ -1055,6 +1055,9 @@ class ChannelType(Type):
> > > > > m.value = info.count
> > > > > info.count = m.value + 1
> > > > > info.messages.append(m)
> > > > > + if m.name in info.messages_byname:
> > > > > + raise Exception("Duplicated message name between
> > > > > %s
> > > > > and %s in channel %s" % (
> > > > > + info.messages_byname[m.name].name, m.name,
> > > > > self.name))
> > > >
> > > > I believe this will repeat twice the same name "between xxx and xxx in
> > > > channel ..". You can only mention the duplicate name once.
> > > > Looks good otherwise.
> > > >
> > >
> > > The exception will stop the program, so only the first is shown
> >
> > It will show only one Exception message yes, but it will be something
> > like
> > "Duplicated message name between foo and foo in channel bar"
> > No need for mentioning 'foo' twice in that message.
> >
> > Christophe
> >
>
> When you are right you are right, changing to
>
> raise Exception("Duplicated message name '%s' in channel '%s'" % (m.name, self.name))
Looks good to me, ack.
Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel