{"comment":{"url":"https://api.github.com/repos/LuboO/iCalendar-parser_PA193_Rteam/comments/14209135","html_url":"https://github.com/LuboO/iCalendar-parser_PA193_Rteam/commit/b75f8bd0e0ef1978b522411cc72da8d756bd75fb#commitcomment-14209135","id":14209135,"user":{"login":"LuboO","id":5676691,"avatar_url":"https://avatars.githubusercontent.com/u/5676691?v=3","gravatar_id":"","url":"https://api.github.com/users/LuboO","html_url":"https://github.com/LuboO","followers_url":"https://api.github.com/users/LuboO/followers","following_url":"https://api.github.com/users/LuboO/following{/other_user}","gists_url":"https://api.github.com/users/LuboO/gists{/gist_id}","starred_url":"https://api.github.com/users/LuboO/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LuboO/subscriptions","organizations_url":"https://api.github.com/users/LuboO/orgs","repos_url":"https://api.github.com/users/LuboO/repos","events_url":"https://api.github.com/users/LuboO/events{/privacy}","received_events_url":"https://api.github.com/users/LuboO/received_events","type":"User","site_admin":false},"position":7,"line":14,"path":"src/parameters/encoding.cpp","commit_id":"b75f8bd0e0ef1978b522411cc72da8d756bd75fb","created_at":"2015-11-05T11:52:57Z","updated_at":"2015-11-05T11:52:57Z","body":"Mainly for consistency of error messages (in other parameters, there is no\nfixed value so no value there would yield \"empty parameter\" and in other\ncode it could yield \"invalid parameter\"\nOn 5 Nov 2015 12:49 pm, \"Ondrej Mosnáček\" <notifications@github.com> wrote:\n\n> I don't see why checking if the value is empty is necessary here (and in\n> other similar parameters). You already check that the value is either\n> \"8BIT\" or \"BASE64\", so an empty value would be rejected anyway.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/LuboO/iCalendar-parser_PA193_Rteam/commit/b75f8bd0e0ef1978b522411cc72da8d756bd75fb#commitcomment-14209077>\n> .\n>\n"}}