mirror of
https://github.com/git/git
synced 2024-08-24 18:26:02 +00:00
![Jeff King](/assets/img/avatar_default.png)
Our parse_blob_buffer() takes a ptr/len combo, just like parse_tree_buffer(), etc, and returns success or failure. But it doesn't actually do anything with them; we just set the "parsed" flag in the object and return success, without even looking at the contents. There could be some value to keeping these unused parameters: - it's consistent with the parse functions for other object types. But we already lost that consistency in837d395a5c
(Replace parse_blob() with an explanatory comment, 2010-01-18). - As the comment from837d395a5c
explains, callers are supposed to make sure they have the object content available. So in theory asking for these parameters could serve as a signal. But there are only two callers, and one of them always passes NULL (after doing a streaming check of the object hash). This shows that there aren't likely to be a lot of callers (since everyone either uses the type-generic parse functions, or handles blobs individually), and that they need to take special care anyway (because we usually want to avoid loading whole blobs in memory if we can avoid it). So let's just drop these unused parameters, and likewise the useless return value. While we're touching the header file, let's move the declaration of parse_blob_buffer() right below that explanatory comment, where it's more likely to be seen by people looking for the function. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
20 lines
417 B
C
20 lines
417 B
C
#include "cache.h"
|
|
#include "blob.h"
|
|
#include "repository.h"
|
|
#include "alloc.h"
|
|
|
|
const char *blob_type = "blob";
|
|
|
|
struct blob *lookup_blob(struct repository *r, const struct object_id *oid)
|
|
{
|
|
struct object *obj = lookup_object(r, oid);
|
|
if (!obj)
|
|
return create_object(r, oid, alloc_blob_node(r));
|
|
return object_as_type(obj, OBJ_BLOB, 0);
|
|
}
|
|
|
|
void parse_blob_buffer(struct blob *item)
|
|
{
|
|
item->object.parsed = 1;
|
|
}
|