3 ms·

Is that a common thing in GUI to have a single function over 900 LoC? https://github.com/casperstorm/ajour/blob/master/src/gui/update.rs#L21 https://github.com

by Thaxll 6y ago

Is that a common thing in GUI to have a single function over 900 LoC? https://github.com/casperstorm/ajour/blob/master/src/gui/update.rs#L21 https://github.com/casperstorm/ajour/blob/master/src/gui/upd... https://github.com/casperstorm/ajour/blob/master/src/gui/element.rs#L27 https://github.com/casperstorm/ajour/blob/master/src/gui/ele...

culinary-robot 6y ago
I am very proud of the code base, but there's many places to optimize. I welcome contributions if you want to help out!
ziml77 6y ago
Ehh it doesn't seem to be a problem. Understanding the function doesn't require thinking about those 900 lines at once since the code for handling each message type doesn't interact with any other code in the function.
stefan_ 6y ago
It's a WndProc, feel free to split each message into a separate function but that's not actually helping anyones understanding.
deleted 6y ago
[deleted]