CircuitPython - a Python implementation for teaching coding with microcontrollers https://git.suspectdevices.com/suspect-devices/circuitpython/atom?h=4.0.0-beta.7 2019-02-16T00:53:19+00:00 2019-02-16T00:53:19+00:00 Scott Shawcroft scott@tannewt.org 2019-02-16T00:53:19+00:00 urn:sha1:e6b140e7a0cf07fa365269e7801bb111565cb478 Fixes #1127 2018-12-03T17:04:32+00:00 Dan Halbert halbert@halwitz.org 2018-12-03T16:00:35+00:00 urn:sha1:80db2cec99ca5c995a0302b1d6d00ad77cfcd736 2018-06-18T02:35:56+00:00 Damien George damien.p.george@gmail.com 2018-06-13T01:54:44+00:00 urn:sha1:6abede2ca9e221b6aefcaccbda0c89e367507df1 The existing mp_get_stream_raise() helper does explicit checks that the input object is a real pointer object, has a non-NULL stream protocol, and has the desired stream C method (read/write/ioctl). In most cases it is not necessary to do these checks because it is guaranteed that the input object has the stream protocol and desired C methods. For example, native objects that use the stream wrappers (eg mp_stream_readinto_obj) in their locals dict always have the stream protocol (or else they shouldn't have these wrappers in their locals dict). This patch introduces an efficient mp_get_stream() which doesn't do any checks and just extracts the stream protocol struct. This should be used in all cases where the argument object is known to be a stream. The existing mp_get_stream_raise() should be used primarily to verify that an object does have the correct stream protocol methods. All uses of mp_get_stream_raise() in py/stream.c have been converted to use mp_get_stream() because the argument is guaranteed to be a proper stream object. This patch improves efficiency of stream operations and reduces code size. 2018-06-04T06:53:17+00:00 Damien George damien.p.george@gmail.com 2018-06-04T06:53:17+00:00 urn:sha1:1427f8f593062a9f428d40e8d3536c6a97adc479 Since a long time now, mp_obj_type_t no longer refers explicitly to mp_stream_p_t but rather to an abstract "const void *protocol". So there's no longer any need to define mp_stream_p_t in obj.h and it can go with all its associated definitions in stream.h. Pretty much all users of this type will already include the stream header. 2018-05-01T05:54:50+00:00 Ayke van Laethem aykevanlaethem@gmail.com 2018-04-24T15:43:14+00:00 urn:sha1:d43c7377564b3e1be36f627a465df2c5779a1ae8 This is a more consistent use of errno codes. For example, it may be that a stream returns MP_EAGAIN but the mp_is_nonblocking_error() macro doesn't catch this value because it checks for EAGAIN instead (which may have a different value than MP_EAGAIN when MICROPY_USE_INTERNAL_ERRNO is enabled). 2018-04-10T03:41:32+00:00 Damien George damien.p.george@gmail.com 2018-03-07T06:48:53+00:00 urn:sha1:cf31d384f1d2ca09f817f154892eaa6860c10144 This patch moves the implementation of stream closure from a dedicated method to the ioctl of the stream protocol, for each type that implements closing. The benefits of this are: 1. Rounds out the stream ioctl function, which already includes flush, seek and poll (among other things). 2. Makes calling mp_stream_close() on an object slightly more efficient because it now no longer needs to lookup the close method and call it, rather it just delegates straight to the ioctl function (if it exists). 3. Reduces code size and allows future types that implement the stream protocol to be smaller because they don't need a dedicated close method. Code size reduction is around 200 bytes smaller for x86 archs and around 30 bytes smaller for the bare-metal archs. 2017-08-20T19:02:41+00:00 Paul Sokolovsky pfalcon@users.sourceforge.net 2017-08-20T18:32:17+00:00 urn:sha1:0cd9ab77550eada4def492a3a25286ead71a3b24 For SEEK_SET, offset should be treated as unsigned, to allow full-width stream sizes (e.g. 32-bit instead of 31-bit). This is now fully documented in stream.h. Also, seek symbolic constants are added. 2017-07-31T08:35:40+00:00 Alexander Steffen devel.20.webmeister@spamgourmet.com 2017-06-30T07:22:17+00:00 urn:sha1:55f33240f3d7051d4213629e92437a36f1fac50e There were several different spellings of MicroPython present in comments, when there should be only one. 2017-07-18T01:57:39+00:00 Alexander Steffen devel.20.webmeister@spamgourmet.com 2017-06-29T21:14:58+00:00 urn:sha1:299bc625864b9e624ed599c94a5f95870516139a The code conventions suggest using header guards, but do not define how those should look like and instead point to existing files. However, not all existing files follow the same scheme, sometimes omitting header guards altogether, sometimes using non-standard names, making it easy to accidentally pick a "wrong" example. This commit ensures that all header files of the MicroPython project (that were not simply copied from somewhere else) follow the same pattern, that was already present in the majority of files, especially in the py folder. The rules are as follows. Naming convention: * start with the words MICROPY_INCLUDED * contain the full path to the file * replace special characters with _ In addition, there are no empty lines before #ifndef, between #ifndef and one empty line before #endif. #endif is followed by a comment containing the name of the guard macro. py/grammar.h cannot use header guards by design, since it has to be included multiple times in a single C file. Several other files also do not need header guards as they are only used internally and guaranteed to be included only once: * MICROPY_MPHALPORT_H * mpconfigboard.h * mpconfigport.h * mpthreadport.h * pin_defs_*.h * qstrdefs*.h 2016-12-02T05:37:29+00:00 Damien George damien.p.george@gmail.com 2016-12-02T05:37:29+00:00 urn:sha1:304cfda8c466920b1d29903bfca8bca5ed110f3b The constants MP_IOCTL_POLL_xxx, which were stmhal-specific, are moved from stmhal/pybioctl.h (now deleted) to py/stream.h. And they are renamed to MP_STREAM_POLL_xxx to be consistent with other such constants. All uses of these constants have been updated.