907c5d387f
Originally, black_bindings found each contiguous "//|" block and sent it to black independently. This was slower than it needed to be. Instead, swap the comment prefix: when running black, take off "//|" prefixes and put "##|" prefixes on all un-prefixed lines. Then, after black is run, do the opposite operation This more than doubles the overall speed of "pre-commit run --all", from 3m20s to 55s CPU time on my local machine (32.5s to under 10s "elapsed" time) It also causes a small amount of churn in the bindings, because black now sees enough context to know whether one 'def' follows another or ends the 'def's in a 'class'. In the latter case, it adds an extra newline, which becomes a "//|" line. I'm less sure why a trailing comma was omitted before down in rp2pio/StateMachine.c but let's roll with it.
70 lines
2.7 KiB
C
70 lines
2.7 KiB
C
/*
|
|
* This file is part of the MicroPython project, http://micropython.org/
|
|
*
|
|
* The MIT License (MIT)
|
|
*
|
|
* Copyright (c) 2019 Dan Halbert for Adafruit Industries
|
|
* Copyright (c) 2018 Artur Pacholec
|
|
* Copyright (c) 2017 Glenn Ruben Bakke
|
|
*
|
|
* Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
* of this software and associated documentation files (the "Software"), to deal
|
|
* in the Software without restriction, including without limitation the rights
|
|
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
* copies of the Software, and to permit persons to whom the Software is
|
|
* furnished to do so, subject to the following conditions:
|
|
*
|
|
* The above copyright notice and this permission notice shall be included in
|
|
* all copies or substantial portions of the Software.
|
|
*
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
|
* THE SOFTWARE.
|
|
*/
|
|
|
|
#include <string.h>
|
|
|
|
#include "py/objproperty.h"
|
|
#include "py/runtime.h"
|
|
#include "shared-bindings/_bleio/ScanResults.h"
|
|
|
|
//| class ScanResults:
|
|
//| """Iterates over advertising data received while scanning. This object is always created
|
|
//| by a `_bleio.Adapter`: it has no user-visible constructor."""
|
|
//|
|
|
STATIC mp_obj_t scanresults_iternext(mp_obj_t self_in) {
|
|
mp_check_self(mp_obj_is_type(self_in, &bleio_scanresults_type));
|
|
bleio_scanresults_obj_t *self = MP_OBJ_TO_PTR(self_in);
|
|
mp_obj_t scan_entry = common_hal_bleio_scanresults_next(self);
|
|
if (scan_entry != mp_const_none) {
|
|
return scan_entry;
|
|
}
|
|
return MP_OBJ_STOP_ITERATION;
|
|
}
|
|
|
|
//| def __init__(self) -> None:
|
|
//| """Cannot be instantiated directly. Use `_bleio.Adapter.start_scan`."""
|
|
//| ...
|
|
//| def __iter__(self) -> Iterator[ScanEntry]:
|
|
//| """Returns itself since it is the iterator."""
|
|
//| ...
|
|
//| def __next__(self) -> ScanEntry:
|
|
//| """Returns the next `_bleio.ScanEntry`. Blocks if none have been received and scanning is still
|
|
//| active. Raises `StopIteration` if scanning is finished and no other results are available."""
|
|
//| ...
|
|
//|
|
|
|
|
const mp_obj_type_t bleio_scanresults_type = {
|
|
{ &mp_type_type },
|
|
.flags = MP_TYPE_FLAG_EXTENDED,
|
|
.name = MP_QSTR_ScanResults,
|
|
MP_TYPE_EXTENDED_FIELDS(
|
|
.getiter = mp_identity_getiter,
|
|
.iternext = scanresults_iternext,
|
|
),
|
|
};
|