From: Bruno Haible <bruno@clisp.org>
To: Daiki Ueno <ueno@gnu.org>
Cc: bug-gnulib@gnu.org
Subject: Re: [PATCH] read-file: add variants that clear internal memory
Date: Thu, 28 May 2020 22:09:59 +0200 [thread overview]
Message-ID: <1715829.BQkuVFqnNA@omega> (raw)
In-Reply-To: <87h7w1bmgw.fsf-ueno@gnu.org>
> Both has been fixed and pushed. Thank you for the review!
Let me update the uses of the module 'read-file' in Gnulib.
I think the next weekly CI run would have caught this.
2020-05-28 Bruno Haible <bruno@clisp.org>
Fix build errors due to read-file changes (regression from 2020-05-27).
* lib/git-merge-changelog.c (read_changelog_file): Update read_file
invocation.
* tests/test-sameacls.c (main): Likewise.
* tests/test-pipe-filter-gi1.c (main): Call read_file instead of
read_binary_file.
* tests/test-pipe-filter-ii1.c (main): Likewise.
diff --git a/lib/git-merge-changelog.c b/lib/git-merge-changelog.c
index 1e6dae1..7b74a49 100644
--- a/lib/git-merge-changelog.c
+++ b/lib/git-merge-changelog.c
@@ -1,5 +1,5 @@
/* git-merge-changelog - git "merge" driver for GNU style ChangeLog files.
- Copyright (C) 2008-2010 Bruno Haible <bruno@clisp.org>
+ Copyright (C) 2008-2020 Bruno Haible <bruno@clisp.org>
This program is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
@@ -300,7 +300,7 @@ read_changelog_file (const char *filename, struct changelog_file *result)
/* Read the file in text mode, otherwise it's hard to recognize empty
lines. */
size_t length;
- char *contents = read_file (filename, &length);
+ char *contents = read_file (filename, 0, &length);
if (contents == NULL)
{
fprintf (stderr, "could not read file '%s'\n", filename);
diff --git a/tests/test-pipe-filter-gi1.c b/tests/test-pipe-filter-gi1.c
index 4ee9375..0994610 100644
--- a/tests/test-pipe-filter-gi1.c
+++ b/tests/test-pipe-filter-gi1.c
@@ -80,7 +80,7 @@ main (int argc, char *argv[])
/* Read some text from a file. */
input_filename = argv[2];
- input = read_binary_file (input_filename, &input_size);
+ input = read_file (input_filename, RF_BINARY, &input_size);
ASSERT (input != NULL);
/* Convert it to uppercase, line by line. */
diff --git a/tests/test-pipe-filter-ii1.c b/tests/test-pipe-filter-ii1.c
index 5f31d37..5a56c55 100644
--- a/tests/test-pipe-filter-ii1.c
+++ b/tests/test-pipe-filter-ii1.c
@@ -102,7 +102,7 @@ main (int argc, char *argv[])
/* Read some text from a file. */
input_filename = argv[2];
- input = read_binary_file (input_filename, &input_size);
+ input = read_file (input_filename, RF_BINARY, &input_size);
ASSERT (input != NULL);
/* Convert it to uppercase, line by line. */
diff --git a/tests/test-sameacls.c b/tests/test-sameacls.c
index cdb10f4..6aad92f 100644
--- a/tests/test-sameacls.c
+++ b/tests/test-sameacls.c
@@ -55,14 +55,14 @@ main (int argc, char *argv[])
size_t size2;
char *contents2;
- contents1 = read_file (file1, &size1);
+ contents1 = read_file (file1, 0, &size1);
if (contents1 == NULL)
{
fprintf (stderr, "error reading file %s: errno = %d\n", file1, errno);
fflush (stderr);
abort ();
}
- contents2 = read_file (file2, &size2);
+ contents2 = read_file (file2, 0, &size2);
if (contents2 == NULL)
{
fprintf (stderr, "error reading file %s: errno = %d\n", file2, errno);
next prev parent reply other threads:[~2020-05-28 20:10 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-26 8:35 [PATCH] read-file: add variants that clear internal memory Daiki Ueno
2020-05-26 10:54 ` Bruno Haible
2020-05-26 16:32 ` Daiki Ueno
2020-05-26 18:33 ` Bruno Haible
2020-05-27 6:43 ` Daiki Ueno
2020-05-27 11:12 ` Bruno Haible
2020-05-27 12:18 ` Daiki Ueno
2020-05-28 20:09 ` Bruno Haible [this message]
2020-05-29 3:59 ` Daiki Ueno
2020-05-29 9:15 ` Bruno Haible
2020-05-29 15:23 ` Daiki Ueno
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://lists.gnu.org/mailman/listinfo/bug-gnulib
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1715829.BQkuVFqnNA@omega \
--to=bruno@clisp.org \
--cc=bug-gnulib@gnu.org \
--cc=ueno@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).