quoted
This looks like a good cleanup, but should be split into another cleanup
commit. It looks unrelated.
Agree.
Will split up in next patchset.
quoted
You've got all sorts of mixed space/tab indent here.
Yes.
Will fix wrong indents in next patchset.
quoted
Isn't accepting http and https the default?
I think it's NOT after I took a view at function `fetch_pack_config`.
This may be optimized in another patch I think and also will listen
to some other suggestions.
Thank you.