1:48:36PuercoPopscottj: yeah I saw from the phone earlier. It is truee that when using find-symbol we should specify the package, but I'm not sure If we should be doing that vs an alist containing the accessors, going to go over the code real quick or merge the proposed fix
2:17:19PuercoPopscottj: What do you think of https://github.com/stumpwm/stumpwm/pull/539
2:20:02travis-cistumpwm/stumpwm#715 (floating-modifier-fix - f372d6d : Javier Olaechea): The build passed.
2:31:12PuercoPopThe more I think about it, the more that just scratching the modifiers structure makes sense. Why use a struct to keep the mapping?
2:31:48PuercoPopIf you are already storing the struct in a global variable
4:36:39scottjPuercoPop: I don't have a preference/opinion either way
4:52:47scottjPuercoPop: I think I prefer not changing all the callers, and if keeping the struct then the association list feels like duplicating data for little benefit over direct slot-value access.
4:53:57scottjI don't have much experience in this area though