diff options
| author | Jeff Layton <jlayton@redhat.com> | 2011-04-05 21:02:37 +0200 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2011-04-12 02:57:12 +0200 |
| commit | 581ade4d1c025eb10421eda0d0c0a2f04447d7c5 (patch) | |
| tree | b65a074f6f3cf769f3add8f59e37758e37e70fca /MAINTAINERS | |
| parent | cifs: clean up length checks in check2ndT2 (diff) | |
| download | linux-581ade4d1c025eb10421eda0d0c0a2f04447d7c5.tar.xz linux-581ade4d1c025eb10421eda0d0c0a2f04447d7c5.zip | |
cifs: clean up various nits in unicode routines (try #2)
Minor revision to the original patch. Don't abuse the __le16 variable
on the stack by casting it to wchar_t and handing it off to char2uni.
Declare an actual wchar_t on the stack instead. This fixes a valid
sparse warning.
Fix the spelling of UNI_ASTERISK. Eliminate the unneeded len_remaining
variable in cifsConvertToUCS.
Also, as David Howells points out. We were better off making
cifsConvertToUCS *not* use put_unaligned_le16 since it means that we
can't optimize the mapped characters at compile time. Switch them
instead to use cpu_to_le16, and simply use put_unaligned to set them
in the string.
Reported-and-acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'MAINTAINERS')
0 files changed, 0 insertions, 0 deletions