[PATCH 4/6] core: add message handler

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

 



On Wed, 2017-10-04 at 14:00 +0300, Tanu Kaskinen wrote:
> On Tue, 2017-10-03 at 19:41 +0200, Georg Chini wrote:
> > I don't
> > mind dropping the _core_ part from the names. I would then also
> > rename pa_core_send_message() to
> > pa_message_handler_send_message() and put it in the same header
> > file.
> 
> I don't like renaming it to pa_message_handler_send_message(), because
> the function isn't a "method" (in OOP lingo) of a message handler. It
> fits better as a method of the core object.

You can ignore this comment. pa_message_handler_send_message() doesn't
feel so wrong to me any more as it initially did. It's nice to keep the
send_message() function together with the message handler functions.

-- 
Tanu

https://www.patreon.com/tanuk


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux