ZheNing Hu <adlternative@xxxxxxxxx> 于2021年6月3日周四 下午1:36写道: > > Junio C Hamano <gitster@xxxxxxxxx> 于2021年6月3日周四 上午10:38写道: > > But I do not think we want to add the new change at this location, > > at least for two reasons: > > > > * The posted patch checks '!arg' to avoid rejecting "raw:size", > > which would not scale at all. What if you wanted to later add > > "raw:upcase", which you must reject? > > > > Yeah, the code here makes "raw" lack of scalability. Especially we > want to add "%(raw:textconv)" and "%(raw:filter)" later. > Now I am building %(raw:textconv) and %(raw:filter), the code will be very difficult to write: if (format->quote_style && !strncmp(sp, "raw", 3) && ((!arg) || (!strncmp(arg, ":textconv", 9)) || (!strncmp(arg, ":filter", 7)))) return strbuf_addf_ret(err, -1, _("--format=%.*s cannot be used with" "--python, --shell, --tcl, --perl"), (int)(ep-atom), atom); Is there any good way? Thanks. -- ZheNing Hu