Fix nxsytle warning

Signed-off-by: Xiang Xiao <xiaoxiang@xiaomi.com>
Change-Id: I36099dc6c07c7ada2f9fcb06fe0267b8d213a61a
This commit is contained in:
Xiang Xiao
2020-07-02 22:28:09 +08:00
committed by Abdelatif Guettouche
parent deaa6c5b7b
commit 942f32e22a
3 changed files with 101 additions and 28 deletions

View File

@@ -167,7 +167,9 @@ static void show_statfs(const char *path)
struct statfs buf; struct statfs buf;
int ret; int ret;
/* Try stat() against a file or directory. It should fail with expectederror */ /* Try stat() against a file or directory. It should fail with
* expectederror
*/
printf("show_statfs: Try statfs(%s)\n", path); printf("show_statfs: Try statfs(%s)\n", path);
ret = statfs(path, &buf); ret = statfs(path, &buf);
@@ -206,9 +208,9 @@ static void show_directories(const char *path, int indent)
int i; int i;
dirp = opendir(path); dirp = opendir(path);
if ( !dirp ) if (!dirp)
{ {
printf("show_directories: ERROR opendir(\"%s\") failed with errno=%d\n", printf("show_directories: ERROR opendir(\"%s\") with errno=%d\n",
path, errno); path, errno);
g_nerrors++; g_nerrors++;
return; return;
@@ -220,13 +222,14 @@ static void show_directories(const char *path, int indent)
{ {
putchar(' '); putchar(' ');
} }
if (DIRENT_ISDIRECTORY(direntry->d_type)) if (DIRENT_ISDIRECTORY(direntry->d_type))
{ {
char *subdir; char *subdir;
printf("%s/\n", direntry->d_name); printf("%s/\n", direntry->d_name);
sprintf(g_namebuffer, "%s/%s", path, direntry->d_name); sprintf(g_namebuffer, "%s/%s", path, direntry->d_name);
subdir = strdup(g_namebuffer); subdir = strdup(g_namebuffer);
show_directories( subdir, indent + 1); show_directories(subdir, indent + 1);
free(subdir); free(subdir);
} }
else else
@@ -261,7 +264,7 @@ static void fail_read_open(const char *path, int expectederror)
} }
else if (errno != expectederror) else if (errno != expectederror)
{ {
printf("fail_read_open: ERROR open(%s) failed with errno=%d (expected %d)\n", printf("fail_read_open: ERROR open(%s) with errno=%d(expect %d)\n",
path, errno, expectederror); path, errno, expectederror);
g_nerrors++; g_nerrors++;
} }
@@ -301,9 +304,10 @@ static void read_test_file(const char *path)
} }
else else
{ {
buffer[127]='\0'; buffer[127] = '\0';
printf("read_test_file: Read \"%s\" from %s\n", buffer, path); printf("read_test_file: Read \"%s\" from %s\n", buffer, path);
} }
close(fd); close(fd);
} }
} }
@@ -320,10 +324,10 @@ static void write_test_file(const char *path)
printf("write_test_file: opening %s for writing\n", path); printf("write_test_file: opening %s for writing\n", path);
fd = open(path, O_WRONLY|O_CREAT|O_TRUNC, 0644); fd = open(path, O_WRONLY | O_CREAT | O_TRUNC, 0644);
if (fd < 0) if (fd < 0)
{ {
printf("write_test_file: ERROR failed to open %s for writing, errno=%d\n", printf("write_test_file: ERROR to open %s for writing, errno=%d\n",
path, errno); path, errno);
g_nerrors++; g_nerrors++;
} }
@@ -340,6 +344,7 @@ static void write_test_file(const char *path)
{ {
printf("write_test_file: wrote %d bytes to %s\n", nbytes, path); printf("write_test_file: wrote %d bytes to %s\n", nbytes, path);
} }
close(fd); close(fd);
} }
} }
@@ -352,7 +357,9 @@ static void fail_mkdir(const char *path, int expectederror)
{ {
int ret; int ret;
/* Try mkdir() against a file or directory. It should fail with expectederror */ /* Try mkdir() against a file or directory. It should fail with
* expectederror
*/
printf("fail_mkdir: Try mkdir(%s)\n", path); printf("fail_mkdir: Try mkdir(%s)\n", path);
@@ -364,7 +371,7 @@ static void fail_mkdir(const char *path, int expectederror)
} }
else if (errno != expectederror) else if (errno != expectederror)
{ {
printf("fail_mkdir: ERROR mkdir(%s) failed with errno=%d (expected %d)\n", printf("fail_mkdir: ERROR mkdir(%s) with errno=%d(expect %d)\n",
path, errno, expectederror); path, errno, expectederror);
g_nerrors++; g_nerrors++;
} }
@@ -397,7 +404,9 @@ static void fail_rmdir(const char *path, int expectederror)
{ {
int ret; int ret;
/* Try rmdir() against a file or directory. It should fail with expectederror */ /* Try rmdir() against a file or directory. It should fail with
* expectederror
*/
printf("fail_rmdir: Try rmdir(%s)\n", path); printf("fail_rmdir: Try rmdir(%s)\n", path);
@@ -409,7 +418,7 @@ static void fail_rmdir(const char *path, int expectederror)
} }
else if (errno != expectederror) else if (errno != expectederror)
{ {
printf("fail_rmdir: ERROR rmdir(%s) failed with errno=%d (expected %d)\n", printf("fail_rmdir: ERROR rmdir(%s) with errno=%d(expect %d)\n",
path, errno, expectederror); path, errno, expectederror);
g_nerrors++; g_nerrors++;
} }
@@ -442,7 +451,9 @@ static void fail_unlink(const char *path, int expectederror)
{ {
int ret; int ret;
/* Try unlink() against a file or directory. It should fail with expectederror */ /* Try unlink() against a file or directory. It should fail with
* expectederror
*/
printf("fail_unlink: Try unlink(%s)\n", path); printf("fail_unlink: Try unlink(%s)\n", path);
@@ -454,7 +465,7 @@ static void fail_unlink(const char *path, int expectederror)
} }
else if (errno != expectederror) else if (errno != expectederror)
{ {
printf("fail_unlink: ERROR unlink(%s) failed with errno=%d (expected %d)\n", printf("fail_unlink: ERROR unlink(%s) with errno=%d(expect %d)\n",
path, errno, expectederror); path, errno, expectederror);
g_nerrors++; g_nerrors++;
} }
@@ -485,11 +496,14 @@ static void succeed_unlink(const char *path)
* Name: fail_rename * Name: fail_rename
****************************************************************************/ ****************************************************************************/
static void fail_rename(const char *oldpath, const char *newpath, int expectederror) static void fail_rename(const char *oldpath, const char *newpath,
int expectederror)
{ {
int ret; int ret;
/* Try rename() against a file or directory. It should fail with expectederror */ /* Try rename() against a file or directory. It should fail with
* expectederror
*/
printf("fail_rename: Try rename(%s->%s)\n", oldpath, newpath); printf("fail_rename: Try rename(%s->%s)\n", oldpath, newpath);
@@ -502,7 +516,7 @@ static void fail_rename(const char *oldpath, const char *newpath, int expecteder
} }
else if (errno != expectederror) else if (errno != expectederror)
{ {
printf("fail_rename: ERROR rename(%s->%s) failed with errno=%d (expected %d)\n", printf("fail_rename: ERROR rename(%s->%s) with errno=%d(expect %d)\n",
oldpath, newpath, errno, expectederror); oldpath, newpath, errno, expectederror);
g_nerrors++; g_nerrors++;
} }
@@ -537,7 +551,9 @@ static void fail_stat(const char *path, int expectederror)
struct stat buf; struct stat buf;
int ret; int ret;
/* Try stat() against a file or directory. It should fail with expectederror */ /* Try stat() against a file or directory. It should fail with
* expectederror
*/
printf("fail_stat: Try stat(%s)\n", path); printf("fail_stat: Try stat(%s)\n", path);
@@ -550,7 +566,7 @@ static void fail_stat(const char *path, int expectederror)
} }
else if (errno != expectederror) else if (errno != expectederror)
{ {
printf("fail_stat: ERROR stat(%s) failed with errno=%d (expected %d)\n", printf("fail_stat: ERROR stat(%s) failed with errno=%d(expected %d)\n",
path, errno, expectederror); path, errno, expectederror);
g_nerrors++; g_nerrors++;
} }
@@ -632,7 +648,9 @@ int main(int argc, FAR char *argv[])
show_statfs(g_testfile1); show_statfs(g_testfile1);
read_test_file(g_testfile1); read_test_file(g_testfile1);
#else #else
/* Create the test directory that would have been on the canned filesystem */ /* Create the test directory that would have been on the canned
* filesystem
*/
succeed_mkdir(g_testdir1); succeed_mkdir(g_testdir1);
show_directories("", 0); show_directories("", 0);
@@ -640,7 +658,9 @@ int main(int argc, FAR char *argv[])
show_statfs(g_testdir1); show_statfs(g_testdir1);
#endif #endif
/* Write a test file into a pre-existing directory on the test file system */ /* Write a test file into a pre-existing directory on the test file
* system
*/
fail_stat(g_testfile2, ENOENT); fail_stat(g_testfile2, ENOENT);
write_test_file(g_testfile2); write_test_file(g_testfile2);
@@ -652,16 +672,22 @@ int main(int argc, FAR char *argv[])
read_test_file(g_testfile2); read_test_file(g_testfile2);
/* Try rmdir() against a file on the directory. It should fail with ENOTDIR */ /* Try rmdir() against a file on the directory. It should fail with
* ENOTDIR
*/
#ifdef CONFIG_EXAMPLES_MOUNT_DEVNAME #ifdef CONFIG_EXAMPLES_MOUNT_DEVNAME
fail_rmdir(g_testfile1, ENOTDIR); fail_rmdir(g_testfile1, ENOTDIR);
#endif #endif
/* Try rmdir() against the test directory. It should fail with ENOTEMPTY */ /* Try rmdir() against the test directory. It should fail with
* ENOTEMPTY
*/
fail_rmdir(g_testdir1, ENOTEMPTY); fail_rmdir(g_testdir1, ENOTEMPTY);
/* Try unlink() against the test directory. It should fail with EISDIR */ /* Try unlink() against the test directory. It should fail with
* EISDIR
*/
fail_unlink(g_testdir1, EISDIR); fail_unlink(g_testdir1, EISDIR);
@@ -676,7 +702,9 @@ int main(int argc, FAR char *argv[])
#ifdef CONFIG_EXAMPLES_MOUNT_DEVNAME #ifdef CONFIG_EXAMPLES_MOUNT_DEVNAME
fail_read_open(g_testfile1, ENOENT); fail_read_open(g_testfile1, ENOENT);
#endif #endif
/* Try rmdir() against the test directory. It should still fail with ENOTEMPTY */ /* Try rmdir() against the test directory. It should still fail with
* ENOTEMPTY
*/
fail_rmdir(g_testdir1, ENOTEMPTY); fail_rmdir(g_testdir1, ENOTEMPTY);
@@ -731,7 +759,7 @@ int main(int argc, FAR char *argv[])
succeed_stat(g_testdir3); succeed_stat(g_testdir3);
show_statfs(g_testdir3); show_statfs(g_testdir3);
/* Try rename() on the root directory. Should fail with EXDEV*/ /* Try rename() on the root directory. Should fail with EXDEV */
fail_rename(g_target, g_testdir4, EXDEV); fail_rename(g_target, g_testdir4, EXDEV);

View File

@@ -1,5 +1,47 @@
/****************************************************************************
* apps/interpreters/minibasic/basic.c
*
* Copyright (C) 2016 Gregory Nutt. All rights reserved.
*
* This file was taken from Mini Basic, versino 1.0 developed by Malcolm
* McLean, Leeds University. Mini Basic version 1.0 was released the
* Creative Commons Attribution license which, from my reading, appears to
* be compatible with the NuttX BSD-style license:
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
* are met:
*
* 1. Redistributions of source code must retain the above copyright
* notice, this list of conditions and the following disclaimer.
* 2. Redistributions in binary form must reproduce the above copyright
* notice, this list of conditions and the following disclaimer in
* the documentation and/or other materials provided with the
* distribution.
* 3. Neither the name NuttX nor the names of its contributors may be
* used to endorse or promote products derived from this software
* without specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
* FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
* COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
* INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
* BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS
* OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
* AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
* ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*
****************************************************************************/
/****************************************************************************
* Included Files
****************************************************************************/
#include <sys/stat.h> #include <sys/stat.h>
#include <sys/statfs.h>
#include <stdio.h> #include <stdio.h>
#include <stdlib.h> #include <stdlib.h>
@@ -7,6 +49,10 @@
#include "ficl.h" #include "ficl.h"
/****************************************************************************
* Public Functions
****************************************************************************/
void *ficlMalloc(size_t size) void *ficlMalloc(size_t size)
{ {
return malloc(size); return malloc(size);

View File

@@ -26,7 +26,6 @@
#include <sys/types.h> #include <sys/types.h>
#include <sys/mount.h> #include <sys/mount.h>
#include <sys/statfs.h>
#include <stdint.h> #include <stdint.h>
#include <stdbool.h> #include <stdbool.h>