diff options
author | Thomas Martitz <kugel@rockbox.org> | 2011-09-07 23:16:29 +0000 |
---|---|---|
committer | Thomas Martitz <kugel@rockbox.org> | 2011-09-07 23:16:29 +0000 |
commit | 5296af838c39b8b7c4b00a3c896345f9e0719dcf (patch) | |
tree | e3e31a974f23bd4cd810c00b892c1eabaec4c676 /firmware/buflib.c | |
parent | 6a989b8d4dec4750ad474598bd52897381d2f3cb (diff) | |
download | rockbox-5296af838c39b8b7c4b00a3c896345f9e0719dcf.tar.gz rockbox-5296af838c39b8b7c4b00a3c896345f9e0719dcf.zip |
Buflib: Clarification about invalid handles
* Enhance allocation function comments to better state the return value and what an invalid value is
* Change clients to check for "< 0" instead of "<= 0" or "== 0"
* Return -1 or -2 depending on the exact failure in buflib_alloc_ex.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@30469 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'firmware/buflib.c')
-rw-r--r-- | firmware/buflib.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/firmware/buflib.c b/firmware/buflib.c index f7ef35ea8b..7c5f3d208e 100644 --- a/firmware/buflib.c +++ b/firmware/buflib.c | |||
@@ -425,7 +425,7 @@ handle_alloc: | |||
425 | goto handle_alloc; | 425 | goto handle_alloc; |
426 | } | 426 | } |
427 | } | 427 | } |
428 | return 0; | 428 | return -1; |
429 | } | 429 | } |
430 | } | 430 | } |
431 | 431 | ||
@@ -471,7 +471,7 @@ buffer_alloc: | |||
471 | } else { | 471 | } else { |
472 | handle->val=1; | 472 | handle->val=1; |
473 | handle_free(ctx, handle); | 473 | handle_free(ctx, handle); |
474 | return 0; | 474 | return -2; |
475 | } | 475 | } |
476 | } | 476 | } |
477 | 477 | ||