* [PATCH 1/1] fs: fat: remove superfluous assignments
@ 2021-01-25 23:14 Heinrich Schuchardt
2021-01-26 7:53 ` Christian Gmeiner
2021-01-30 19:21 ` Tom Rini
0 siblings, 2 replies; 4+ messages in thread
From: Heinrich Schuchardt @ 2021-01-25 23:14 UTC (permalink / raw)
To: u-boot
Do not assign a value to a variable if it is not used.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
fs/fat/fat.c | 3 ++-
fs/fat/fat_write.c | 6 +++---
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/fs/fat/fat.c b/fs/fat/fat.c
index fb6ce094ac..ccba268f61 100644
--- a/fs/fat/fat.c
+++ b/fs/fat/fat.c
@@ -248,7 +248,6 @@ static __u32 get_fatent(fsdata *mydata, __u32 entry)
static int
get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
{
- __u32 idx = 0;
__u32 startsect;
int ret;
@@ -277,6 +276,8 @@ get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
size -= mydata->sect_size;
}
} else {
+ __u32 idx;
+
idx = size / mydata->sect_size;
if (idx == 0)
ret = 0;
diff --git a/fs/fat/fat_write.c b/fs/fat/fat_write.c
index aae3a6a3d1..b43a27b205 100644
--- a/fs/fat/fat_write.c
+++ b/fs/fat/fat_write.c
@@ -573,7 +573,6 @@ static __u32 determine_fatent(fsdata *mydata, __u32 entry)
static int
set_sectors(fsdata *mydata, u32 startsect, u8 *buffer, u32 size)
{
- u32 nsects = 0;
int ret;
debug("startsect: %d\n", startsect);
@@ -595,6 +594,8 @@ set_sectors(fsdata *mydata, u32 startsect, u8 *buffer, u32 size)
size -= mydata->sect_size;
}
} else if (size >= mydata->sect_size) {
+ u32 nsects;
+
nsects = size / mydata->sect_size;
ret = disk_write(startsect, nsects, buffer);
if (ret != nsects) {
@@ -785,7 +786,6 @@ get_set_cluster(fsdata *mydata, __u32 clustnum, loff_t pos, __u8 *buffer,
}
size -= wsize;
- buffer += wsize;
*gotsize += wsize;
}
@@ -1482,10 +1482,10 @@ static int delete_single_dentry(fat_itr *itr)
*/
static int delete_long_name(fat_itr *itr)
{
- struct dir_entry *dent = itr->dent;
int seqn = itr->dent->nameext.name[0] & ~LAST_LONG_ENTRY_MASK;
while (seqn--) {
+ struct dir_entry *dent;
int ret;
ret = delete_single_dentry(itr);
--
2.29.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 1/1] fs: fat: remove superfluous assignments
2021-01-25 23:14 [PATCH 1/1] fs: fat: remove superfluous assignments Heinrich Schuchardt
@ 2021-01-26 7:53 ` Christian Gmeiner
2021-01-26 15:10 ` Heinrich Schuchardt
2021-01-30 19:21 ` Tom Rini
1 sibling, 1 reply; 4+ messages in thread
From: Christian Gmeiner @ 2021-01-26 7:53 UTC (permalink / raw)
To: u-boot
Hi
Am Di., 26. Jan. 2021 um 00:14 Uhr schrieb Heinrich Schuchardt
<xypron.glpk@gmx.de>:
>
> Do not assign a value to a variable if it is not used.
I am not very happy about this commit message tbo. What you are doing
is reducing the scope
of variables.. so write this in your commit message.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> fs/fat/fat.c | 3 ++-
> fs/fat/fat_write.c | 6 +++---
> 2 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/fs/fat/fat.c b/fs/fat/fat.c
> index fb6ce094ac..ccba268f61 100644
> --- a/fs/fat/fat.c
> +++ b/fs/fat/fat.c
> @@ -248,7 +248,6 @@ static __u32 get_fatent(fsdata *mydata, __u32 entry)
> static int
> get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
> {
> - __u32 idx = 0;
> __u32 startsect;
> int ret;
>
> @@ -277,6 +276,8 @@ get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
> size -= mydata->sect_size;
> }
> } else {
> + __u32 idx;
> +
> idx = size / mydata->sect_size;
> if (idx == 0)
> ret = 0;
> diff --git a/fs/fat/fat_write.c b/fs/fat/fat_write.c
> index aae3a6a3d1..b43a27b205 100644
> --- a/fs/fat/fat_write.c
> +++ b/fs/fat/fat_write.c
> @@ -573,7 +573,6 @@ static __u32 determine_fatent(fsdata *mydata, __u32 entry)
> static int
> set_sectors(fsdata *mydata, u32 startsect, u8 *buffer, u32 size)
> {
> - u32 nsects = 0;
> int ret;
>
> debug("startsect: %d\n", startsect);
> @@ -595,6 +594,8 @@ set_sectors(fsdata *mydata, u32 startsect, u8 *buffer, u32 size)
> size -= mydata->sect_size;
> }
> } else if (size >= mydata->sect_size) {
> + u32 nsects;
> +
> nsects = size / mydata->sect_size;
> ret = disk_write(startsect, nsects, buffer);
> if (ret != nsects) {
> @@ -785,7 +786,6 @@ get_set_cluster(fsdata *mydata, __u32 clustnum, loff_t pos, __u8 *buffer,
> }
>
> size -= wsize;
> - buffer += wsize;
That looks wrong when staring at it from my mail client.
> *gotsize += wsize;
> }
>
> @@ -1482,10 +1482,10 @@ static int delete_single_dentry(fat_itr *itr)
> */
> static int delete_long_name(fat_itr *itr)
> {
> - struct dir_entry *dent = itr->dent;
> int seqn = itr->dent->nameext.name[0] & ~LAST_LONG_ENTRY_MASK;
>
> while (seqn--) {
> + struct dir_entry *dent;
> int ret;
>
> ret = delete_single_dentry(itr);
> --
> 2.29.2
>
--
greets
--
Christian Gmeiner, MSc
https://christian-gmeiner.info/privacypolicy
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/1] fs: fat: remove superfluous assignments
2021-01-26 7:53 ` Christian Gmeiner
@ 2021-01-26 15:10 ` Heinrich Schuchardt
0 siblings, 0 replies; 4+ messages in thread
From: Heinrich Schuchardt @ 2021-01-26 15:10 UTC (permalink / raw)
To: u-boot
On 26.01.21 08:53, Christian Gmeiner wrote:
> Hi
>
> Am Di., 26. Jan. 2021 um 00:14 Uhr schrieb Heinrich Schuchardt
> <xypron.glpk@gmx.de>:
>>
>> Do not assign a value to a variable if it is not used.
>
> I am not very happy about this commit message tbo. What you are doing
> is reducing the scope
> of variables.. so write this in your commit message.
Sure.
>
>>
>> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>> ---
>> fs/fat/fat.c | 3 ++-
>> fs/fat/fat_write.c | 6 +++---
>> 2 files changed, 5 insertions(+), 4 deletions(-)
>>
>> diff --git a/fs/fat/fat.c b/fs/fat/fat.c
>> index fb6ce094ac..ccba268f61 100644
>> --- a/fs/fat/fat.c
>> +++ b/fs/fat/fat.c
>> @@ -248,7 +248,6 @@ static __u32 get_fatent(fsdata *mydata, __u32 entry)
>> static int
>> get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
>> {
>> - __u32 idx = 0;
>> __u32 startsect;
>> int ret;
>>
>> @@ -277,6 +276,8 @@ get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
>> size -= mydata->sect_size;
>> }
>> } else {
>> + __u32 idx;
>> +
>> idx = size / mydata->sect_size;
>> if (idx == 0)
>> ret = 0;
>> diff --git a/fs/fat/fat_write.c b/fs/fat/fat_write.c
>> index aae3a6a3d1..b43a27b205 100644
>> --- a/fs/fat/fat_write.c
>> +++ b/fs/fat/fat_write.c
>> @@ -573,7 +573,6 @@ static __u32 determine_fatent(fsdata *mydata, __u32 entry)
>> static int
>> set_sectors(fsdata *mydata, u32 startsect, u8 *buffer, u32 size)
>> {
>> - u32 nsects = 0;
>> int ret;
>>
>> debug("startsect: %d\n", startsect);
>> @@ -595,6 +594,8 @@ set_sectors(fsdata *mydata, u32 startsect, u8 *buffer, u32 size)
>> size -= mydata->sect_size;
>> }
>> } else if (size >= mydata->sect_size) {
>> + u32 nsects;
>> +
>> nsects = size / mydata->sect_size;
>> ret = disk_write(startsect, nsects, buffer);
>> if (ret != nsects) {
>> @@ -785,7 +786,6 @@ get_set_cluster(fsdata *mydata, __u32 clustnum, loff_t pos, __u8 *buffer,
>> }
>>
>> size -= wsize;
>> - buffer += wsize;
>
> That looks wrong when staring at it from my mail client.
This is the last time buffer is accessed before return. Why do you think
it is wrong?
Best regards
Heinrich
>
>> *gotsize += wsize;
>> }
>>
>> @@ -1482,10 +1482,10 @@ static int delete_single_dentry(fat_itr *itr)
>> */
>> static int delete_long_name(fat_itr *itr)
>> {
>> - struct dir_entry *dent = itr->dent;
>> int seqn = itr->dent->nameext.name[0] & ~LAST_LONG_ENTRY_MASK;
>>
>> while (seqn--) {
>> + struct dir_entry *dent;
>> int ret;
>>
>> ret = delete_single_dentry(itr);
>> --
>> 2.29.2
>>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/1] fs: fat: remove superfluous assignments
2021-01-25 23:14 [PATCH 1/1] fs: fat: remove superfluous assignments Heinrich Schuchardt
2021-01-26 7:53 ` Christian Gmeiner
@ 2021-01-30 19:21 ` Tom Rini
1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2021-01-30 19:21 UTC (permalink / raw)
To: u-boot
On Tue, Jan 26, 2021 at 12:14:14AM +0100, Heinrich Schuchardt wrote:
> Do not assign a value to a variable if it is not used.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210130/0dd3a803/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-01-30 19:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-25 23:14 [PATCH 1/1] fs: fat: remove superfluous assignments Heinrich Schuchardt
2021-01-26 7:53 ` Christian Gmeiner
2021-01-26 15:10 ` Heinrich Schuchardt
2021-01-30 19:21 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.