yamt commented on a change in pull request #1280:
URL: https://github.com/apache/incubator-nuttx/pull/1280#discussion_r445961455
##########
File path: include/stdio.h
##########
@@ -225,6 +216,49 @@ FILE *popen(FAR const char *command, FAR const char *mode);
int pclose(FILE *stream);
#endif
+/* These APIs are not implemented and/or can be synthesized from
+ * supported APIs.
+ */
+
+#ifdef CONFIG_HAVE_INLINE
+static inline int putc(int c, FAR FILE *s)
Review comment:
i can understand the performance argument for ctype functions.
but i'm not sure about these FILE* functions. have you measured?
if you really care the (IMO marginal) performance gain,
i think the traditional convention is to have a macro version overriding the
non-inline function
so that any app which is not happy with the macro version can do "#undef
getc" before using it.
like:
int getc(FAR FILE *stream);
#define getc(s) fgetc(s)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
