Re: [Outreachy] [PATCH] blame: Convert pickaxe_blame defined constants to enums

2019-10-15 Thread Wambui Karuga
On Mon, Oct 14, 2019 at 02:46:29PM -0700, Jonathan Tan wrote: > > > Well, I agree that it could be better, but with the C language as we > > > have it now, I still slightly prefer an enum to a list of #define. Both > > > ways, we still have to manually enter values for each flag, but with > > > enu

Re: [Outreachy] [PATCH] blame: Convert pickaxe_blame defined constants to enums

2019-10-14 Thread Wambui Karuga
d" or "int"). gdb supports the notion that a few people use > enums this way too, but if we decide as a project to not use enums in > this way, that's fine too. For what it's worth, I tried doing a search > online, but most of the results I got was for C# (where it is > recommended - they have a "[Flags]" attribute for enums), so maybe I am > indeed in the minority. I'll try to pick another set of constants to convert - before this is agreed on. Thanks, wambui karuga

Re: [Outreachy] [PATCH] blame: Convert pickaxe_blame defined constants to enums

2019-10-11 Thread Wambui Karuga
t; being immediately after the "#includes" which is why I moved them, but I'll > > try to stick to the > > convention from now on. > > Thanks for providing your justifications. > > > > Also, I have a slight preference for putting "= 02" on the BLAME_COPY > > > line but that is not necessary. > > > > > Noted. > > Well, Junio provides a good reason for putting "= 02" [1], so please do > that. > > [1] https://public-inbox.org/git/xmqqsgnzj4vs@gitster-ct.c.googlers.com/ Yes, I'll include it in an updated patch Thanks wambui karuga

Re: [Outreachy] [PATCH] blame: Convert pickaxe_blame defined constants to enums

2019-10-11 Thread Wambui Karuga
On Thu, Oct 10, 2019 at 11:44:39AM -0700, Jonathan Tan wrote: > > Convert pickaxe_blame preprocessor constants in blame.h to an enum. > > Also replace previous instances of the constants with the new enum values. > > First of all, thanks for your initiative in finding a microproject and > making a

[Outreachy] [PATCH] blame: Convert pickaxe_blame defined constants to enums

2019-10-10 Thread Wambui Karuga
Convert pickaxe_blame preprocessor constants in blame.h to an enum. Also replace previous instances of the constants with the new enum values. Signed-off-by: Wambui Karuga --- blame.c | 8 blame.h | 12 +++- builtin/blame.c | 17 - 3 files

[Outreachy] Outreachy initial contact

2019-10-10 Thread Wambui Karuga
reachy and I found a list of good first issues to begin with. I'm looking forward to working on one of the most used system in open source! Wambui Karuga