[PATCH 1/8] misc: Introduce a struct osmo_pcu and move things into it.

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-net-gprs@lists.osmocom.org/.

Ivan Kluchnikov Ivan.Kluchnikov at fairwaves.ru
Tue Jul 30 09:17:36 UTC 2013


2013/7/30 Holger Hans Peter Freyther <hfreyther at sysmocom.de>

> I wanted to differentiate that 'pcu' is not a local variable. I could
> have picked g_pcu or something else. I used 'the' because there is only
> one pcu right now. I will remove that limit though (and remove global
> state from the PCU).
>

Ok, I understand.


>
> > Now I am ready for new patchset.
>
> Thanks. I will send things tomorrow.
>
>
> > What is the reasonable amount of time for patchset review for you?
> > Sometimes I am overloaded by other work, but I will try to delay review
> no
> > longer than 1-2 days.
>
> 'reasonable' anything < 5 days. I just want to avoid ending with 40
> patches and you as the maintainer not having time to look at them. E.g.
> for OpenBSC I try to at least give a quick NACK/or ask for changes.
>

 I think we should use this mode, because as you mentioned, reviews are the
good opportunity to catch mistakes.



-- 
Regards,
Ivan Kluchnikov.
http://fairwaves.ru
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/osmocom-net-gprs/attachments/20130730/2c5cdf5c/attachment.htm>


More information about the osmocom-net-gprs mailing list