[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Subject: [PATCH] explicit cast to char*


On 8 March 2017 at 16:38, Maximilian Noppel <max@noppelmax.online> wrote:
>
>> Based on your patches seems like you're using a C++ compiler for these
>> C files, which generally is a bad idea.
>
> I did; But its a good idea not to use 'new' as a var name anyway, i think.
>
> The cast also does make the code better, imho.
>
if you look around you'll find that both topics are very subjective ;-)
That aside patches are valid, even if they are bit unusual.

-Emil


References:
Subject: [PATCH] explicit cast to char*Max Noppel <max@noppelmax.online>
Re: Subject: [PATCH] explicit cast to char*Emil Velikov <emil.l.velikov@xxxxxxxxx>
Re: Subject: [PATCH] explicit cast to char*Maximilian Noppel <max@noppelmax.online>