freemangordon | Wizzup: latest abook master saves to personal addressbook, evolution correctly sees the created contact :) | 00:57 |
---|---|---|
Wizzup | that's sweet | 01:01 |
Wizzup | I guess we can sync as well using syncevo | 01:01 |
Wizzup | (I can sync my fremantle contacts at least) | 01:01 |
Wizzup | exciting :) | 01:43 |
lel | MerlijnWajer synchronize a pull request: https://github.com/maemo-leste/connui-internet/pull/2 (WIP: add providers tab to advanced iap settings) | 01:45 |
Wizzup | parazyd: btw, if the service_id becomes an unknown value to the icd2 provider (i.e. a config is deleted from tor cpa), then the network will no longer be marked as service provider | 01:47 |
Wizzup | Since the libicd plugin will reject it based on the id not being supported, and then UI acts appropriately | 01:47 |
Wizzup | so we actually don't need to worry about that use case :) | 01:47 |
parazyd | Wizzup: That's pretty useful :) | 07:21 |
Wizzup | freemangordon: any idea if text added to a gtk combo box (marked const char*) is copied to the widget itself? | 07:31 |
Wizzup | I would hope so, but not sure | 07:31 |
parazyd | Wizzup: In what sense? | 07:34 |
Wizzup | looks like it is fine | 07:34 |
Wizzup | parazyd: I wasn't sure if I had to g_strdup() strings I add with gtk_combo_box_append_text | 07:34 |
Wizzup | but it looks I don't need to | 07:34 |
parazyd | No you don't really | 07:35 |
Wizzup | as in, I didn't allocate them (gconf did) | 07:35 |
Wizzup | yeah | 07:35 |
Wizzup | it makes sense I guess | 07:35 |
parazyd | mhm | 07:35 |
lel | MerlijnWajer synchronize a pull request: https://github.com/maemo-leste/connui-internet/pull/2 (WIP: add providers tab to advanced iap settings) | 07:52 |
Wizzup | freemangordon: ^^ if you have a couple of minutes | 07:53 |
lel | MerlijnWajer edited a pull request: https://github.com/maemo-leste/connui-internet/pull/2 (Add providers tab to advanced iap settings) | 07:55 |
lel | MerlijnWajer synchronize a pull request: https://github.com/maemo-leste/connui-internet/pull/2 (Add providers tab to advanced iap settings) | 07:57 |
sicelo | freemangordon: yay! that's beautiful | 08:23 |
freemangordon | Wizzup: please fix block brackets :) | 13:46 |
freemangordon | actually the whole PR styling needs to be fixed | 13:46 |
Wizzup | freemangordon: ok, apart from indent? | 13:47 |
freemangordon | see https://github.com/maemo-leste/connui-internet/pull/2/commits/feb8316ca434d858e3c0b9381b7526dafa33bd98#diff-f5f00673365a45906cdb8c63dedef131c800df08c9b1d70e940a7cd933742fdcR67 and https://github.com/maemo-leste/connui-internet/pull/2/commits/feb8316ca434d858e3c0b9381b7526dafa33bd98#diff-f5f00673365a45906cdb8c63dedef131c800df08c9b1d70e940a7cd933742fdcR77 | 13:48 |
freemangordon | the second has the function name on the same line as function type | 13:48 |
freemangordon | brackets have to be on a separate line | 13:48 |
Wizzup | yeah, I'll make it match yours | 13:48 |
freemangordon | mhm | 13:49 |
Wizzup | but I mean apart from indent | 13:49 |
Wizzup | sorry, we're just used to different things :D | 13:49 |
Wizzup | but it's absolutely np, I'll fix it | 13:49 |
freemangordon | one question: why gtk_combo_box_new_text() ? | 13:49 |
Wizzup | As opposed to / instead of? | 13:49 |
freemangordon | I am almost sure some hildon widget is used instead | 13:49 |
freemangordon | lemme try to find it | 13:49 |
Wizzup | I think I took this from other combo boxes from connui | 13:50 |
freemangordon | hmm, could be | 13:50 |
* freemangordon checks | 13:50 | |
Wizzup | The only limitation it has that I don't want to fix right now is that when you switch providers to something else and then back to what it was, it won't "remember" the service id (i.e. provider specific config) you selected initially, but I think that shouldn't be a big deal | 13:51 |
uvos | oh btw please rewiev the mce and h-d pr | 13:51 |
freemangordon | uvos: will do | 13:52 |
freemangordon | Wizzup: I am almost sure hildon_picker_button/hildon_touch_selector is used elsewhere | 13:54 |
Wizzup | do you have an example? I took this from the wlan tab | 13:54 |
Wizzup | in advanced | 13:55 |
freemangordon | oh, right | 13:55 |
freemangordon | pickers are on the main 'page' | 13:55 |
freemangordon | "Internet connections" that is | 13:56 |
freemangordon | so yeah, combobox is fine | 13:56 |
freemangordon | LMK when you fix the coding style | 13:56 |
freemangordon | uvos: reviewing those will take more time I have ATM, will try to do h-d at least tomorrow | 13:57 |
uvos | mce is way more important | 13:57 |
uvos | is blocking everything | 13:57 |
freemangordon | ok | 13:58 |
uvos | the hildon pr just replaces a hack i wrote some time ago with soemthing better | 13:58 |
freemangordon | I am not sure how usefull I will be, given I never used cmake :) | 13:58 |
uvos | so it dosent change any real functionality | 13:58 |
freemangordon | but will do my best | 13:58 |
uvos | thank you | 13:58 |
freemangordon | Wizzup: also, please keep 80 chars per line limit | 14:17 |
uvos | freemangordon: maybe you want to way in on https://github.com/maemo-leste/bugtracker/issues/329 | 14:18 |
uvos | freemangordon: at least for new things the clear consensus so far was linux style | 14:18 |
uvos | (which includes relaxed 80 col limit since some time) | 14:19 |
freemangordon | uvos: this is not new | 14:19 |
Wizzup | he did weigh in on it uvos :p | 14:19 |
Wizzup | https://github.com/maemo-leste/bugtracker/issues/329#issuecomment-594787405 | 14:19 |
uvos | well if he has nothing further to add we need to close 329 by voteing on a style or something | 14:20 |
uvos | something to put into guidelines | 14:20 |
Wizzup | I think the concensus we have is that we stick to whatever the repo has | 14:20 |
Wizzup | and if the maintainer wants to switch, they can do that | 14:20 |
uvos | thats not how 329 reads | 14:21 |
freemangordon | :nod: | 14:21 |
Wizzup | maybe we can narrow it down to like two or three styles | 14:21 |
Wizzup | and then make it clear which uses which | 14:21 |
uvos | and besides a prefered style is sensible | 14:21 |
Wizzup | not if we can't agree on one :P | 14:21 |
uvos | even if it comes with a dont change coherent repos preface | 14:21 |
Wizzup | I'm fine changing my PR for fmg's style, but I won't use it for new repos (only I) work on | 14:21 |
bencoh | :) | 14:22 |
freemangordon | well, at the end of the day I think it comes to which style is the easiest to read | 14:22 |
uvos | i dont think there is a style that is easiest to read | 14:22 |
uvos | that just whatever you are used to | 14:22 |
Wizzup | I will switch to your style if you switch to my editor ;) | 14:23 |
freemangordon | not really | 14:23 |
Wizzup | (joke) | 14:23 |
bencoh | acctually a 80-ish limit really is easier to read | 14:23 |
freemangordon | Wizzup: and give up on QtCreator? | 14:23 |
Wizzup | :p | 14:23 |
uvos | i dissagree on modern displays that just creates whitespace | 14:23 |
bencoh | it basically means your eyes to need to move too far | 14:23 |
freemangordon | bencoh: :nod: | 14:23 |
uvos | to many line breaks is bad too | 14:23 |
Wizzup | I just want to get to a sitation where we can use tooling with a set of indentation rules | 14:23 |
bencoh | and that you can have several opened files in columns | 14:23 |
uvos | sure | 14:23 |
Wizzup | and then you can run it when you work, revert it with the standard rules, and commit code | 14:23 |
uvos | but 80 coll is to narrow imo | 14:24 |
uvos | 110 at least | 14:24 |
uvos | 200 is def to long | 14:24 |
uvos | etc | 14:24 |
bencoh | (s/to need/don't need/ ) | 14:24 |
freemangordon | uvos: no matter the display, eyes still have limit on the minimum font size that's ok for them | 14:24 |
freemangordon | and that changes with aging | 14:24 |
bencoh | uvos: oh, right, the limit itself is obviously a matter of taste, and 80 is legacy | 14:24 |
uvos | right | 14:24 |
uvos | its very mutch a matter of taste | 14:25 |
freemangordon | 80 chars limit forces you to beautify the code you write, more or less | 14:25 |
bencoh | relaxed-limit in linux is 80 soft-limit, ~100 hard-limit now, right? | 14:25 |
uvos | 110 | 14:25 |
bencoh | freemangordon: indeed :) | 14:25 |
uvos | soft ish at 110 | 14:25 |
freemangordon | that's why I prefer to stick to it | 14:25 |
uvos | 80 is totaly gohne from docs iirc | 14:25 |
bencoh | oh? sad | 14:26 |
bencoh | tbh I always wonder how people manage to keep under 80 with tabsize 8 (I use 4) | 14:26 |
uvos | wel linux uses tab | 14:27 |
uvos | so that counts as one | 14:27 |
freemangordon | which is a huge waste of space IMO | 14:27 |
Wizzup | maybe we can pick 2 rules and settle on those, and make some bash aliases or helper scripts to let people switch from styles before they commit? | 14:27 |
uvos | tab? | 14:27 |
bencoh | 8 ? definitely a waste | 14:27 |
Wizzup | as much as I like discussing code formatting | 14:27 |
freemangordon | yes | 14:27 |
Wizzup | I don't think this leading us anywhere | 14:27 |
freemangordon | Wizzup: ok | 14:27 |
bencoh | Wizzup: didn't you toy with indent (the tool) at some point? | 14:27 |
uvos | how is tab a waste of space you can set it to any with you like | 14:27 |
freemangordon | Wizzup: we are really just chattering IIUC | 14:27 |
Wizzup | the one thing my above suggestion requires is if we pick 2 or 3 cdoe styles, we also apply them to a repo, so that subsequent runs do not generate daditional noise | 14:27 |
Wizzup | ok | 14:27 |
Wizzup | then keep going :P | 14:27 |
freemangordon | :) | 14:28 |
bencoh | :] | 14:28 |
uvos | bencoh: indent -linux -l 120 had some consus on the bug above | 14:28 |
bencoh | yeah, having a few reference styles would be a good idea (and I'm all with "stick-to-the-original-repo-choice" thing) | 14:28 |
freemangordon | Wizzup: but, seriously, what we *must* have as a rule is - do PRs in coding style of the repo you're working with | 14:28 |
Wizzup | freemangordon: yes, but I want the style to be strictly defined | 14:29 |
freemangordon | ah, I see | 14:29 |
uvos | sometimes the codeing style the repo works with is hard to decern | 14:29 |
Wizzup | so if you can find a set of indent(1) flags for your style, that would be nice | 14:29 |
uvos | we need to fix that too | 14:29 |
uvos | h-d is an offender | 14:29 |
freemangordon | h-d is... | 14:29 |
freemangordon | Wizzup: I am using QtCreator, not really sure how to extract the style from there | 14:30 |
Wizzup | so my suggestion was, let's try to define 2 or 3 sets of indent flags | 14:30 |
Wizzup | freemangordon: maybe try running indent with some flags until it matches | 14:30 |
Wizzup | it doesn't have to *now*, but it would make our lives easier | 14:30 |
freemangordon | ok | 14:30 |
bencoh | freemangordon: I didn't think you would use it, interesting (actually I think you told me once already, but still surprised) | 14:30 |
freemangordon | that's the best C/C++ editor IMO | 14:31 |
freemangordon | Wizzup: I can export its settings in some qt3.xml file | 14:31 |
bencoh | and yeah, we might not run the full project through indent, but at least people pushing commits would have a way to fix indent in some automatic manner before commiting | 14:31 |
Wizzup | bencoh: yes | 14:31 |
Wizzup | also we can fix the entire project ina single commit and make git blame ignore it | 14:32 |
bencoh | how so? git log/blame/whatever read some config file stored in repository? | 14:32 |
Wizzup | yes | 14:33 |
Wizzup | see the issue linked | 14:33 |
bencoh | oh ... it's semi-automatic then | 14:33 |
freemangordon | https://pastebin.com/TnfF0zAD | 14:34 |
bencoh | interesting | 14:34 |
bencoh | (but only works for blame) | 14:34 |
bencoh | (I guess?) | 14:34 |
Wizzup | bencoh: I think so | 14:34 |
Wizzup | That's also really the only thing that matters | 14:34 |
* freemangordon tries indent | 14:35 | |
freemangordon | Wizzup: can't make it work properly | 17:12 |
freemangordon | it inserts space after * on half of the occasions :( | 17:13 |
freemangordon | like: | 17:18 |
freemangordon | -osso_abook_contact_dispose(GObject *object) | 17:18 |
freemangordon | +osso_abook_contact_dispose(GObject * object) | 17:18 |
Wizzup | freemangordon: maybe there is a flag for that? | 17:24 |
freemangordon | there is | 17:24 |
freemangordon | and it is set | 17:24 |
freemangordon | -par | 17:24 |
freemangordon | but it is ignored | 17:25 |
Wizzup | I don't see that flag in my man page | 17:25 |
freemangordon | --pointer-align-right | 17:26 |
bencoh | I don't see that flag either, funny | 17:27 |
freemangordon | in leste it is there | 17:28 |
Wizzup | freemangordon: it should be the default behaviour | 17:29 |
Wizzup | it looks like | 17:29 |
freemangordon | mhm | 17:30 |
freemangordon | but still a space is insterted afte * | 17:30 |
freemangordon | ncs seems to be ignored too | 17:33 |
freemangordon | -ncs that is | 17:33 |
freemangordon | this is what I tried with: | 17:38 |
freemangordon | indent -di0 -bad -bap -lp -bli0 -l80 -ncs -npcs -nprs -ts8 -nbs -i2 -par -ip0 -as -slc -nut -hnl -ci4 osso-abook-contact.c | 17:38 |
freemangordon | and it breaks it badly | 17:38 |
uvos | uff that style is impossible to read | 18:17 |
uvos | (for me anyhow) | 18:17 |
Wizzup | which is why we can automate it away | 18:17 |
uvos | wierd about the * | 18:18 |
uvos | i dont think i have seen that error | 18:18 |
tmlind_ | uvos: fyi, just pushed out https://github.com/tmlind/linux/commits/droid4-pending-v5.14 and https://github.com/tmlind/linux_openpvrsgx/commits/droid4-pending-pvr-omapdrm-v5.14 | 21:47 |
tmlind_ | seems to work for me based on brief testing | 21:47 |
* tmlind_ sleepy, later | 22:00 |
Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!