Skip to content

Commit

Permalink
Merge pull request #2 from hanchiang/feature/parser-improvement
Browse files Browse the repository at this point in the history
Feature/parser improvement
  • Loading branch information
hanchiang authored May 1, 2022
2 parents 47c0aa8 + 4ec02e4 commit 9fd0dc9
Show file tree
Hide file tree
Showing 4 changed files with 577 additions and 2 deletions.
5 changes: 3 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -125,5 +125,6 @@ $-1


# TODO
* Better separation of concern: Move parse and respond out of ClientInput
* Improve the parsing of user input. Currently, the program only parses an of array of bulk strings, and ignores the bytes part(i.e. $4 in $4\r\nping\r\n) in the bulk string request
- [ ] Better separation of concern: Move parse and respond out of ClientInput
- [ ] Improve the parsing of user input. Currently, the program only parses an of array of bulk strings, and ignores the bytes part(i.e. $4 in $4\r\nping\r\n) in the bulk string request
- [ ] Replace threads with event loop
1 change: 1 addition & 0 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ use std::net::TcpStream;

pub mod request_response;
pub mod store;
pub mod parser;

use crate::request_response::client_input::HandleClientInput;
use request_response::client_input::ClientInput;
Expand Down
1 change: 1 addition & 0 deletions src/parser/mod.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
pub mod parser;
Loading

0 comments on commit 9fd0dc9

Please sign in to comment.