Jakub Narebski <jnareb@xxxxxxxxx> writes: > Set $hash parameter to $hash_base || "HEAD" if it is not set (if it is > not true to be more exact). This allows [hand-edited] URLs with 'action' > "commit" or "commitdiff" but without 'hash' parameter. > > Signed-off-by: Jakub Narebski <jnareb@xxxxxxxxx> > --- > RFC because I want to ask if we should default to HEAD if hash is not > provided for commit and commitdiff views, or should we error out with > more reasonable error message. Probably the latter; the user might be hand-crafting a URL (maybe learned a commit object name from mailing list and cutting and pasting) and mispasted the long hexadecimal string. Silently giving HEAD may leave the user confused than "oops, we do not see that commit object". - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html