|
| 1 | +--- |
| 2 | +title: "GSoC: Week 3" |
| 3 | +date: 2024-06-17 |
| 4 | +description: "" |
| 5 | +type: post |
| 6 | +weight: 25 |
| 7 | +showTableOfContents: false |
| 8 | +--- |
| 9 | + |
| 10 | +## What did I do in this period? |
| 11 | + |
| 12 | +During this week, I reiterated on `t-oid-array` which I mentioned in my last blog. |
| 13 | +In my last blog, I also mentioned that without the_repository->hash_algo being initialized, |
| 14 | +it will cause segfault when using _oid_array_lookup()_. To make reviewing easier and to track the removal |
| 15 | +of _the_repository_, [Patrick](https://gitlab.com/pks-gitlab) introduced a macro USE_THE_REPOSITORY_VARIABLE, and without defining |
| 16 | +it, we cannot access _the_repository_ global variable. In the same series, Patrick also ameneded |
| 17 | +`helper/test-oid-array.c` and `t0064-oid-array.sh` to work without having an actual repository and |
| 18 | +manually initializing _the_hash_algo_ via _repo_set_hash()_. I am waiting for this series ([ps/use-the-repository](https://lore.kernel.org/git/cover.1718347699.git.ps@pks.im/)) to get |
| 19 | +merged to next, and after that I will rebase on top of it to sent `t-oid-array` migration to the list. |
| 20 | + |
| 21 | +Apart from this, I also migrated `helper/test-oidmap.c` to `unit-tests/t-oidmap.c`, and will send to the list |
| 22 | +in the coming week. Both `t-oid-array` and `t-oidmap` rely on `lib-oid` introduced with `t-oidtree`. |
| 23 | + |
| 24 | +In sum, it was a quite week and I did not face any difficulties. |
| 25 | + |
| 26 | +## So what is the plan ahead? |
| 27 | + |
| 28 | +After sending `t-oid-array` and `t-oidmap` to the list, I will start working on `lib-repo` (probably). |
| 29 | +Apart from that `oidset.h` can also take advantage of the new `lib-oid`, but currently there are no existing |
| 30 | +tests for it. If I have some spare time, I'll write some from scratch as the API is similar to `oid-array.h` and `oidmap.h`, so |
| 31 | +it would not take much time. But I'll discuss with mentors first before doing so. |
| 32 | + |
| 33 | +Thanks and see ya' next time! |
0 commit comments