Skip to content

Commit

Permalink
Comment and dependency clearnup
Browse files Browse the repository at this point in the history
  • Loading branch information
nsrCodes committed Mar 17, 2022
1 parent d8aeefd commit 5a3cfc5
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ edition = "2021"

[dependencies]
clap = { version = "3.1.2", features = ["derive"] }
x11rb = { version = "0.9.0", features = ["cursor"] }
x11rb = { version = "0.9.0"}
scrap = "0.5.0"
v4l = {version = "0.12.1", features = ["v4l2-sys"] }
gstreamer = "0.17.4"
Expand Down
1 change: 0 additions & 1 deletion src/options.rs
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ impl SType {

pub fn get_target_path(&self, filename: &String) -> String {
match self {
// todo: convert this into a function
SType::Record => paths::get_video_path(filename)
.as_path()
.display()
Expand Down
4 changes: 2 additions & 2 deletions src/recorder.rs
Original file line number Diff line number Diff line change
Expand Up @@ -175,8 +175,8 @@ impl Media for Recorder {
Element::link_many(&[&src_audio, &raw_audio_caps, &queue_audio, &encoder_audio])
.expect("unable to link audio elements in recording pipeline");
// Linking tail elements
queue_video.link(&muxer).unwrap(); // Video to muxer // TODO (probably overcomlicating): use `link_pad` with sync handler
encoder_audio.link(&muxer).unwrap(); // Audio to muxer // TODO (probably overcomlicating): use `link_pad` with sync handler
queue_video.link(&muxer).unwrap(); // Video to muxer // TODO (probably overcomplicating): use `link_pad` with sync handler
encoder_audio.link(&muxer).unwrap(); // Audio to muxer // TODO (probably overcomplicating): use `link_pad` with sync handler
Element::link_many(&[&muxer, &sink])
.expect("unable to link audio elements in recording pipeline");

Expand Down

0 comments on commit 5a3cfc5

Please sign in to comment.