Greetings to everyone who can't be in Berlin these days!
The third day of the OsmoDevCon 2016 is on. So far we have heard inspiring
talks [1], had a chance to socialize in person, and hacked on using the Osmocom
3G stack in-the-making with a low-cost Iuh capable femto cell, among other
things; usually gathering at 10 a.m., some of us are seen on-location at
IN-Berlin's premises [2] up to midnight. We've just heard the introduction on
Globalstar satellite telephony, and are heading into lunch break any minute
now.
Let me share the quotes of the day! the context being documentation charts and
white-on-black terminals, the quotes as such do provide a surprisingly accurate
description of the preferred dress code of one of our most prominent members:
I hate colors.
-- Harald Welte, 2016-04-23, 11:31 a.m.
I always find white extremely disturbing.
-- Harald Welte, 2016-04-23, 11:38 a.m.
All the best from IN-Berlin and OsmoDevCon 2016!
~Neels
[1] http://projects.osmocom.org/projects/openbsc/wiki/OsmoDevCon2016
[2] http://in-berlin.de/
--
- Neels Hofmeyr <nhofmeyr(a)sysmocom.de> http://www.sysmocom.de/
=======================================================================
* sysmocom - systems for mobile communications GmbH
* Alt-Moabit 93
* 10559 Berlin, Germany
* Sitz / Registered office: Berlin, HRB 134158 B
* Geschäftsführer / Managing Directors: Holger Freyther, Harald Welte
Hi,
I subscribed myself to more change notifications on another gerrit to see how the coments would look like. there is not a lot o context but at least some.
> Begin forwarded message:
>
> Jesus Fernandez has posted comments on this change.
>
> Change subject: Possible crash in QMakeSourceFileInfo
> ......................................................................
>
>
> Patch Set 7:
>
> (6 comments)
>
> Thank you
>
> ....................................................
> File qmake/generators/makefiledeps.cpp
> Line 606
> It's really weird, but you're right.
>
> Dropped... I mean Done.
>
>
> Line 648
> Done
>
>
> Line 655
> Done
>
>
> Line 667
> Done
>
>
> Line 592: for (x += 1 ; x + inc_len < buffer_len && buffer[x + inc_len] != '<';
> And an extra whitespace :)
>
> Done!
>
>
> Line 639: for (x += 1; x + inc_len < buffer_len && buffer[x + inc_len] != '<';
> Done
From: Max <msuraev(a)sysmocom.de>
There is implicit invariant in trx_phy_instance() which is actively used
by various hw-specific implementations to get TRX's phy instance. Let's
make sure there's explicit assertion for this because there's been
segfaults in the past related to it.
---
include/osmo-bts/phy_link.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/osmo-bts/phy_link.h b/include/osmo-bts/phy_link.h
index a559aa3..edc6cc0 100644
--- a/include/osmo-bts/phy_link.h
+++ b/include/osmo-bts/phy_link.h
@@ -125,6 +125,7 @@ void phy_user_statechg_notif(struct phy_instance *pinst, enum phy_link_state lin
static inline struct phy_instance *trx_phy_instance(struct gsm_bts_trx *trx)
{
+ OSMO_ASSERT(trx);
return trx->role_bts.l1h;
}
--
2.8.1