From 39a495a94d0a6bb74d778c2f62162ad10c50c23b Mon Sep 17 00:00:00 2001 From: Joseph Nuthalapati Date: Sat, 5 Oct 2019 12:42:40 +0530 Subject: [PATCH] Change everything to camelCase Signed-off-by: Joseph Nuthalapati --- cli_tube.nim | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/cli_tube.nim b/cli_tube.nim index d7a3d91..2c29825 100644 --- a/cli_tube.nim +++ b/cli_tube.nim @@ -13,32 +13,32 @@ import sequtils, sugar # Supported video players in order of preference # TODO Should go into a config file -let supported_players = @["mpv", "mplayer", "vlc"] +let supportedPlayers = @["mpv", "mplayer", "vlc"] -proc find_supported_player(): string = +proc findSupportedPlayer(): string = for player in supported_players: - let player_bin = execProcess("which " & player) - if player_bin.len != 0: - return strip(player_bin) + let playerBin = execProcess("which " & player) + if playerBin.len != 0: + return strip(playerBin) -proc get_youtube_page(search_query: string): string = - let query_param = encodeUrl(search_query) +proc getYoutubePage(searchQuery: string): string = + let queryParam = encodeUrl(searchQuery) var client = newHttpClient() - let response = get(client, "https://www.youtube.com/results?hl=en&search_query=" & query_param) + let response = get(client, "https://www.youtube.com/results?hl=en&search_query=" & queryParam) # TODO Get rid of temp file or make one temp file per user writeFile("/tmp/cli-tube-page.html", response.body) return "/tmp/cli-tube-page.html" -proc extract_titles_and_urls(html_file: string): seq[tuple[title: string, url: string]] = - loadHtml(html_file).findAll("a"). +proc extractTitlesAndUrls(htmlFile: string): seq[tuple[title: string, url: string]] = + loadHtml(htmlFile).findAll("a"). filter(a => "watch" in a.attrs["href"] and a.attrs.hasKey "title"). map(a => (a.attrs["title"], "https://www.youtube.com" & a.attrs["href"])) -proc present_video_options(titles_and_urls: seq[tuple[title: string, url: string]]) = - for number, (title, url) in titles_and_urls: +proc presentVideoOptions(titlesAndUrls: seq[tuple[title: string, url: string]]) = + for number, (title, url) in titlesAndUrls: echo number, ". ", title, "\n", url, "\n" -present_video_options( - extract_titles_and_urls( - get_youtube_page( +presentVideoOptions( + extractTitlesAndUrls( + getYoutubePage( paramStr(1)))) -- 2.43.0