lemunozm / ruscii

Terminal graphics engine: build your games in the terminal!
Apache License 2.0
171 stars 15 forks source link

Pencil::new_one should probably be Pencil::clone #8

Closed pumken closed 1 year ago

pumken commented 1 year ago

Pencil::new_one seems to be written as a Clone implementation. For clarity, it should probably be moved to a trait implementation of Clone.

(Other changes to the file were stylistic changes made by a linter.)

lemunozm commented 1 year ago

Totally agree with this change