Use std::io::BufWriter to achieve more flexibility #4

Closed
opened 2022-10-10 20:26:54 +00:00 by giulio-Joshi · 2 comments
giulio-Joshi commented 2022-10-10 20:26:54 +00:00 (Migrated from github.com)

Hello everyone!

I found your awesome project in the Hacktoberfest Discord, I hope my contribution can be of help.

I'm unsure in which direction the project can be expanded, so I attempted at making it more flexible in general: using a BufWriter to apply Inversion of Control on the generate_ascii function.

You can check the code on my branch:
https://github.com/giulio-Joshi/aarty/tree/bufferedOutput

Before barging in with a PR directly, I opted for asking some feedback about it.

Thanks a lot for your time 👍🏻

Hello everyone! I found your awesome project in the Hacktoberfest Discord, I hope my contribution can be of help. I'm unsure in which direction the project can be expanded, so I attempted at making it more flexible in general: using a `BufWriter` to apply Inversion of Control on the `generate_ascii` function. You can check the code on my branch: https://github.com/giulio-Joshi/aarty/tree/bufferedOutput Before barging in with a PR directly, I opted for asking some feedback about it. Thanks a lot for your time 👍🏻
anas-elgarhy commented 2022-10-10 21:33:58 +00:00 (Migrated from github.com)

I liked your edits, and it faster than original method that's I use

I liked your edits, and it faster than original method that's I use
anas-elgarhy commented 2022-10-10 21:35:36 +00:00 (Migrated from github.com)

I have learned new ways to think from you, thanks

I have learned new ways to think from you, thanks
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: anas-elgarhy/aarty#4
No description provided.