Fixes to the seq_file document
On Friday 2008-03-28 19:20, Jonathan Corbet wrote:
>commit 9756ccfda31b4c4544aa010aacf71b6672d668e8
>Date: Fri Mar 28 11:19:56 2008 -0600
>
> Add the seq_file documentation
patch on top:
- add const qualifiers
- remove void* casts
- use proper specifier (%Ld is not valid)
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Signed-off-by: Jan Engelhardt <jengelh@computergmbh.de>
diff --git a/Documentation/filesystems/seq_file.txt b/Documentation/filesystems/seq_file.txt
index 92975ee..cc6cdb9 100644
--- a/Documentation/filesystems/seq_file.txt
+++ b/Documentation/filesystems/seq_file.txt
@@ -107,8 +107,8 @@
static void *ct_seq_next(struct seq_file *s, void *v, loff_t *pos)
{
- loff_t *spos = (loff_t *) v;
- *pos = ++(*spos);
+ loff_t *spos = v;
+ *pos = ++*spos;
return spos;
}
@@ -127,8 +127,8 @@
static int ct_seq_show(struct seq_file *s, void *v)
{
- loff_t *spos = (loff_t *) v;
- seq_printf(s, "%Ld\n", *spos);
+ loff_t *spos = v;
+ seq_printf(s, "%lld\n", (long long)*spos);
return 0;
}
@@ -136,7 +136,7 @@
seq_file iterator is finished by creating a seq_operations structure with
the four functions we have just defined:
- static struct seq_operations ct_seq_ops = {
+ static const struct seq_operations ct_seq_ops = {
.start = ct_seq_start,
.next = ct_seq_next,
.stop = ct_seq_stop,
@@ -204,7 +204,7 @@
static int ct_open(struct inode *inode, struct file *file)
{
return seq_open(file, &ct_seq_ops);
- };
+ }
Here, the call to seq_open() takes the seq_operations structure we created
before, and gets set up to iterate through the virtual file.
@@ -219,7 +219,7 @@
all implemented by the seq_file code itself. So a virtual file's
file_operations structure will look like:
- static struct file_operations ct_file_ops = {
+ static const struct file_operations ct_file_ops = {
.owner = THIS_MODULE,
.open = ct_open,
.read = seq_read,