On Wed, Mar 2, 2016 at 3:21 AM, Sidhant Sharma [:tk] <[email protected]> wrote: > + struct option options[] = { > + OPT__QUIET(&quiet, N_("quiet")), > + OPT_HIDDEN_BOOL(0, "stateless-rpc", &stateless_rpc, NULL), > + OPT_HIDDEN_BOOL(0, "advertise-refs", &advertise_refs, NULL), > + OPT_HIDDEN_BOOL(0, "reject-thin-pack-for-testing", > &reject_thin, NULL), > + OPT_END() > + };
If the patch is already final, don't bother. If not, I think I prefer to keep all these options visible (except the "...-for-testing"). This command is never executed directly by mere mortals. The ones that run it need to know all about these hidden tricks because they're implementing new transports. Another side note. I'm not so sure if we should N_() and _() strings in this command (same reasoning, the command's user is very likely developers, not true users). But it does not harm to i18n-ize the command either. Slightly more work for translators, of course. -- Duy -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
