search

tremor-rs / tremor-runtime

Main Tremor Project Rust Codebase
https://www.tremor.rs
Apache License 2.0
851 stars 126 forks source link

Shrink large functions #20

Open Licenser opened 4 years ago

Licenser commented 4 years ago

We have a number of functions that are marked as clippy::too_many_lines - it would be good to go over them and see if they can be cut down into easier to understand functions.

0xdeafbeef commented 4 years ago

Nothing is found with clippy. Everything is refactored?

anupdhml commented 4 years ago

@0xd34b33f there's quite a few of them still: https://github.com/wayfair-tremor/tremor-runtime/search?q=too_many_lines&unscoped_q=too_many_lines

We have clippy exceptions for them right now so they don't show up with clippy.

If you are interested in refactoring them, feel free to take this!