ifreund changed the topic of #river to: river - a dynamic tiling wayland compositor || https://github.com/riverwm/river || channel logs: https://libera.irclog.whitequark.org/river/
lxsameer has quit [Ping timeout: 260 seconds]
vaivis has quit [Read error: Connection reset by peer]
waleee has quit [Ping timeout: 250 seconds]
notzmv has joined #river
snakedye has quit [Ping timeout: 246 seconds]
vaivis has joined #river
<novakane> ifreund: nice, I'll try to implements the new libwayland function in my clients to see how it goes
<novakane> never saw that there was an @errorName, that's better than having the enum name
vaivis has quit [Read error: Connection reset by peer]
vaivis has joined #river
lxsameer has joined #river
<novakane> ifreund: one question, why do you use snake case function name in Lock.zig?
<ifreund> novakane: got spoiled working on tigerbeetle... I just prefer snake case. Got my fingers crossed that I can convince andrew to make that change to the official style guide
<novakane> ifreund: get you, I'll also prefer snake case, more readable, so I hope you can convince him to change it too :D
<ifreund> compare the readability of putAssumeCapacityNoClobber() and put_assume_capacity_no_clobber() at smaller font size or from farther away
<novakane> yeah I think that the only thing I really hate in the style guide :/
<ifreund> novakane: also @errorName(error.Foo) is just Foo
<ifreund> I just don't trust std.fmt and don't want it to print "error.Foo"
talismanick has quit [Ping timeout: 240 seconds]
<novakane> that nice, I'll use it, not a fan of error.Foo in log message
<novakane> ifreund: and btw, I'm not usually a fan of c_int but I vote in favor of it in the display functions
vaivis has quit [Ping timeout: 246 seconds]
snakedye has joined #river
<ifreund> another option there would be to make the functions return the errno enum from zig's std directly
<ifreund> I don't think I've ever seen any code actually use the "number of things processed" that libwayland returns on success
<novakane> yeah that would works too, would make code less verbose if errno is already in the bindings, I'd just like to have a switch instead of using try
<novakane> both are fine for me, both are better that what we currently have imo
<novakane> so looking at my code, I think I would chose to return the value of os.getErrno(), seems like the cleaner things todo
<ifreund> a Zig error set would be better really, but figuring out all possible errno values libwayland can return for a given function isn't really feasiable
<novakane> yeah that why the errno enum is probably the best choice imo
notzmv has quit [Ping timeout: 260 seconds]
snakedye has quit [Read error: Connection reset by peer]
snakedye has joined #river
vaivis has joined #river
waleee has joined #river
vaivis has quit [Read error: Connection reset by peer]
groknull has joined #river
groknull has quit [Remote host closed the connection]
waleee has quit [Quit: WeeChat 3.5]
lxsameer has quit [Ping timeout: 246 seconds]
pkap has joined #river
pkap has quit [Quit: Client closed]
vaivis has joined #river
pkap has joined #river
dbuckley has quit [Ping timeout: 250 seconds]
dbuckley has joined #river
<pkap> ifreund: I'm quite happy with my PR for layout-pinned mappings now. There is however an open question I'm thinking about. Not sure if this is important to you (or others).
<pkap> It's a bit complicated so I tried to explain it in detail in this comment: https://github.com/riverwm/river/pull/577/files#diff-8281ee4cb120abe2f08f3b8f08b7a807874bc63fc7151348535ef405732d111eR366
<pkap> I'd appreciate your thoughs on this (Doesn't have to be now of course)
<pkap> And anybody else too, of course :)
<ifreund> pkap: cool, thanks for writing that up. Hopefully we can finally get that PR merged soon
<ifreund> I'm off to the bouldering gym now though, catch y'all later o7
<pkap> Have fun ifreund :)
<novakane> now that we have some keyboard expert someone gonna finish the IME PR instead of me :P
<pkap> novakane: I hope you don't mean me when you speak of expertise '=D
<novakane> pkap: not at all, I was thinking about pmkap on github... :P
waleee has joined #river
<pkap> novakane: :D Is the IME thing implemented in sway or are you the first trying to do this?
<novakane> pkap: it is in sway, my PR is based a lot on sway implementation, just missing the keyboard grab part
<pkap> novakane: I see. What client do I need to test this?
<novakane> pkap: I used to test with https://github.com/tadeokondrak/anthywl and foot, just launching anthywl and in foot iirc writing `ta` would start it
<pkap> novakane: Thanks, I'll check it out!
<novakane> couldn't make it works with the current implementation of the keyboard in river, but then I was pretty lost in it xD
talismanick has joined #river
notzmv has joined #river
pkap60 has joined #river
lxsameer has joined #river
pkap has quit [Ping timeout: 252 seconds]
pkap60 is now known as pkap
vaivis has quit [Read error: Connection reset by peer]
pkap has quit [Quit: Client closed]
pkap has joined #river
notzmv has quit [Ping timeout: 240 seconds]
pkap has quit [Quit: Client closed]
<tleydxdy[m]> does fcitx5 work with wayland?
<tleydxdy[m]> if so I can test too
talismanick has quit [Read error: Connection reset by peer]
lxsameer has quit [Ping timeout: 240 seconds]
<novakane> not sure if fctix5 implement the Wayland protocol, I know it works if you use it with GTK apps but not sure in something like foot
<NickH> I have fctix5 working in gtk apps but not in foot.
<tleydxdy[m]> I have it working on xwayland right now, I just make everything I need to type stuff in run under X :)