mhuusko5/PhFacebook — comment on commit c44c4354ab8dbd714375abbea647e13c8b8a3919
File:  line 0
By philippec:

Good catch. In that case, one would want to delete the kFBLoginURL constant since it's no longer referenced anywhere.

Before I accept this pull request, however, I would like to see you match the bracing style of the rest of the file and use a simple, one-line comment in the else case, such as "// Unable to get an access token, show the UI".

And of course delete the constant.

You may put the detailed explanation in the git commit log, but since it is fairly long I would recommend a simple one line comment to explain the gist of the change (like the comment above), then a blank line, then your paragraph. You may want to spell-check the message, there's at least one typo (succesful).

Thanks!