Bug 42954

Summary:	Call: Why does MediaDescription.Reject return a reason?
Product:	Telepathy	Reporter:	Olivier Crête <olivier.crete>
Component:	tp-spec	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED WONTFIX	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium	Keywords:	patch
Version:	git master
Hardware:	Other
OS:	All
Whiteboard:	Call
i915 platform:		i915 features:
Attachments:	patch to just remove the reason on Reject

Description Olivier Crête 2011-11-15 10:32:52 UTC

That bit just doesn't make sense to me.. My guess is that it was supposed to be an "in" parameter. But even then, the reason is always the same, "we don't support these codecs".

Comment.

I propose just reverting patch 3f962ace757d8d323effd5d316c5d5c391c6a698 .. and removing the FIXME.

Comment 1 Olivier Crête 2011-12-13 13:15:22 UTC

It also doesn't make sense as a in parameter.. There is a single reason to reject codecs.. they're not compatible.

Comment 2 Olivier Crête 2011-12-14 13:26:22 UTC

Created attachment 54438 [details] [review]
patch to just remove the reason on Reject

There is no point in having an argument, I don't see another reason..

Comment 3 Xavier Claessens 2011-12-16 04:25:09 UTC

Can't you also reject a MD for example because you don't like the codec? like if it propose animated jpeg and you want to reject even if you could use it? dunno, just wondering...

I would just like avoid having later a RejectWithReason() extra call just because we didn't though about a case... But if you're 100% sure it is useless, I'm ok :)

Comment 4 Olivier Crête 2011-12-16 07:30:09 UTC

Good point, lets keep it then

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.