Stefan Beller <sbeller@xxxxxxxxxx> writes:
> On Mon, Oct 19, 2015 at 12:28 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
>> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx>
>> ---
>> builtin/mailinfo.c | 45 ++++++++++++++++++++++++++-------------------
>> 1 file changed, 26 insertions(+), 19 deletions(-)
>>
>> diff --git a/builtin/mailinfo.c b/builtin/mailinfo.c
>> index 2c194da..ec65805 100644
>> --- a/builtin/mailinfo.c
>> +++ b/builtin/mailinfo.c
>> @@ -7,6 +7,8 @@
>> #include "utf8.h"
>> #include "strbuf.h"
>>
>> +#define MAX_BOUNDARIES 5
>> +
>> struct mailinfo {
>> FILE *input;
>> FILE *output;
>> @@ -22,6 +24,8 @@ struct mailinfo {
>> int use_inbody_headers; /* defaults to 1 */
>> const char *metainfo_charset;
>>
>> + struct strbuf *content[MAX_BOUNDARIES];
>> + struct strbuf **content_top;
>> struct strbuf charset;
>> char *message_id;
>> enum {
>> @@ -35,7 +39,6 @@ struct mailinfo {
>> };
>>
>> #define MAX_HDR_PARSED 10
>> -#define MAX_BOUNDARIES 5
>
> Would it make sense to also move MAX_HDR_PARSED, such that we have all
> defines in one place?
Looking at their final resting place, I do not think so.
--
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