Tuesday, 2023-07-25

T42<TheVancedGamer> mal: checked logcat, no errors08:46
*** rdr_ is now known as rdr16:53
*** rdr_ is now known as rdr17:11
piggzwondering if sailfish will move to https://gitlab.com/ubports/development/core/hybris-support/pulseaudio-module-droid-discover ?17:49
piggzmal: ^^17:49
malpiggz: hmm, need to think17:54
T42<TheVancedGamer> mal: made a quick hack to pa-droid for parsing sampling rates with spaces, how does it look?18:43
T42<TheVancedGamer> https://paste.myself5.de/sibejixono.cpp18:43
mal@TheVancedGamer did you check the package I mentioned? dummy af18:52
T42<TheVancedGamer> mal: yeah dummy is installed but never started19:31
mal@TheVancedGamer try to get it started19:42
maland see what happens19:42
malafaik it's needed for call audio with qti audiosystem-passthrough19:43
T42<TheVancedGamer> mal: hmm, might try in morning19:43
mal@TheVancedGamer https://github.com/mer-hybris/droid-config-sony-lena/blob/master/patterns/patterns-sailfish-device-adaptation-pdx213.inc#L3719:43
T42<TheVancedGamer> could you maybe look at that crude diff in the meanwhile?19:43
maldoes camera app work for your?19:43
T42<TheVancedGamer> works on Volla X23 at least19:43
maleven video recording?19:44
T42<TheVancedGamer> mal: yup, everything works19:44
T42<TheVancedGamer> (halium, not sfos :p)19:44
T42<TheVancedGamer> only thing remaining is call audio and that's it19:44
malbecause I think the error you had might be due to the dummy af not being there so the dbus call fails, or something19:45
maljust a theory at the moment19:45
T42<TheVancedGamer> mal: I'll try that tomorrow morning and see if it works19:47
malok19:47
T42<TheVancedGamer> also was playing around with pa-droid and tried adding parsing support for spaced sampling rates19:47
T42<TheVancedGamer> https://paste.myself5.de/sibejixono.cpp19:48
T42<TheVancedGamer> if you wanna look19:48
malany example of such rate?19:50
mal@TheVancedGamer about that change, would it make more sense to have some local delimiter variable and testing the input for space and VALUE_SEPARATOR to see which is used then doing the loop using the selected one, less code duplication that way19:57
mal*the existing loop19:58
T42<TheVancedGamer> I feel like I can do better, just don't know how I could20:00
T42<TheVancedGamer> mal: Volla X23 has audio policy version 720:00
T42<TheVancedGamer> which uses spaces20:00
T42<TheVancedGamer> works there20:00
malsomething like "char separator = strchr(str, " ") ? ' ' : VALUE_SEPARATOR;" and then "while ((entry = pa_split(str, VALUE_SEPARATOR, &state))) {" -> "while ((entry = pa_split(str, separator, &state))) {"20:02
malwouldn't that work in theory?20:02
T42<TheVancedGamer> mal: yeah that could20:05

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!