Skip to content

Commit e1af819

Browse files
jeplerjepler
committed
fix (unused) default for some required args
1 parent 085cf0e commit e1af819

7 files changed

Lines changed: 14 additions & 14 deletions

File tree

‎shared-bindings/audiocore/RawSample.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
STATIC mp_obj_t audioio_rawsample_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *all_args) {
7373
enum { ARG_buffer, ARG_channel_count, ARG_sample_rate };
7474
static const mp_arg_t allowed_args[] = {
75-
{ MP_QSTR_buffer, MP_ARG_OBJ | MP_ARG_REQUIRED },
75+
{ MP_QSTR_buffer, MP_ARG_OBJ | MP_ARG_REQUIRED, {.u_obj = MP_OBJ_NULL } },
7676
{ MP_QSTR_channel_count, MP_ARG_INT | MP_ARG_KW_ONLY, {.u_int = 1 } },
7777
{ MP_QSTR_sample_rate, MP_ARG_INT | MP_ARG_KW_ONLY, {.u_int = 8000} },
7878
};

‎shared-bindings/audiomixer/Mixer.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ MP_PROPERTY_GETTER(audiomixer_mixer_voice_obj,
201201
STATIC mp_obj_t audiomixer_mixer_obj_play(size_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) {
202202
enum { ARG_sample, ARG_voice, ARG_loop };
203203
static const mp_arg_t allowed_args[] = {
204-
{ MP_QSTR_sample, MP_ARG_OBJ | MP_ARG_REQUIRED },
204+
{ MP_QSTR_sample, MP_ARG_OBJ | MP_ARG_REQUIRED, {} },
205205
{ MP_QSTR_voice, MP_ARG_INT | MP_ARG_KW_ONLY, {.u_int = 0} },
206206
{ MP_QSTR_loop, MP_ARG_BOOL | MP_ARG_KW_ONLY, {.u_bool = false} },
207207
};

‎shared-bindings/audiomixer/MixerVoice.c‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ STATIC mp_obj_t audiomixer_mixervoice_make_new(const mp_obj_type_t *type, size_t
6464
STATIC mp_obj_t audiomixer_mixervoice_obj_play(size_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) {
6565
enum { ARG_sample, ARG_loop };
6666
static const mp_arg_t allowed_args[] = {
67-
{ MP_QSTR_sample, MP_ARG_OBJ | MP_ARG_REQUIRED },
67+
{ MP_QSTR_sample, MP_ARG_OBJ | MP_ARG_REQUIRED, {} },
6868
{ MP_QSTR_loop, MP_ARG_BOOL | MP_ARG_KW_ONLY, {.u_bool = false} },
6969
};
7070
audiomixer_mixervoice_obj_t *self = MP_OBJ_TO_PTR(pos_args[0]);
@@ -105,7 +105,7 @@ MP_DEFINE_CONST_FUN_OBJ_1(audiomixer_mixervoice_get_level_obj, audiomixer_mixerv
105105
STATIC mp_obj_t audiomixer_mixervoice_obj_set_level(size_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) {
106106
enum { ARG_level };
107107
static const mp_arg_t allowed_args[] = {
108-
{ MP_QSTR_level, MP_ARG_OBJ | MP_ARG_REQUIRED },
108+
{ MP_QSTR_level, MP_ARG_OBJ | MP_ARG_REQUIRED, {} },
109109
};
110110
audiomixer_mixervoice_obj_t *self = MP_OBJ_TO_PTR(pos_args[0]);
111111
mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)];

‎shared-bindings/bitmaptools/__init__.c‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -970,11 +970,11 @@ STATIC mp_obj_t bitmaptools_obj_draw_circle(size_t n_args, const mp_obj_t *pos_a
970970
enum {ARG_dest_bitmap, ARG_x, ARG_y, ARG_radius, ARG_value};
971971

972972
static const mp_arg_t allowed_args[] = {
973-
{MP_QSTR_dest_bitmap, MP_ARG_REQUIRED | MP_ARG_OBJ},
974-
{MP_QSTR_x, MP_ARG_REQUIRED | MP_ARG_INT},
975-
{MP_QSTR_y, MP_ARG_REQUIRED | MP_ARG_INT},
976-
{MP_QSTR_radius, MP_ARG_REQUIRED | MP_ARG_INT},
977-
{MP_QSTR_value, MP_ARG_REQUIRED | MP_ARG_INT},
973+
{MP_QSTR_dest_bitmap, MP_ARG_REQUIRED | MP_ARG_OBJ, {}},
974+
{MP_QSTR_x, MP_ARG_REQUIRED | MP_ARG_INT, {}},
975+
{MP_QSTR_y, MP_ARG_REQUIRED | MP_ARG_INT, {}},
976+
{MP_QSTR_radius, MP_ARG_REQUIRED | MP_ARG_INT, {}},
977+
{MP_QSTR_value, MP_ARG_REQUIRED | MP_ARG_INT, {}},
978978
};
979979
mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)];
980980
mp_arg_parse_all(n_args, pos_args, kw_args, MP_ARRAY_SIZE(allowed_args), allowed_args, args);

‎shared-bindings/struct/__init__.c‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,8 @@ MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(struct_unpack_obj, 2, 3, struct_unpack);
132132
STATIC mp_obj_t struct_unpack_from(size_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) {
133133
enum { ARG_format, ARG_buffer, ARG_offset };
134134
static const mp_arg_t allowed_args[] = {
135-
{ MP_QSTR_format, MP_ARG_REQUIRED | MP_ARG_OBJ },
136-
{ MP_QSTR_buffer, MP_ARG_REQUIRED | MP_ARG_OBJ },
135+
{ MP_QSTR_format, MP_ARG_REQUIRED | MP_ARG_OBJ, {} },
136+
{ MP_QSTR_buffer, MP_ARG_REQUIRED | MP_ARG_OBJ, {} },
137137
{ MP_QSTR_offset, MP_ARG_INT, {.u_int = 0} },
138138
};
139139

‎shared-bindings/synthio/MidiTrack.c‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,8 @@
7575
STATIC mp_obj_t synthio_miditrack_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *all_args) {
7676
enum { ARG_buffer, ARG_tempo, ARG_sample_rate, ARG_waveform, ARG_envelope };
7777
static const mp_arg_t allowed_args[] = {
78-
{ MP_QSTR_buffer, MP_ARG_OBJ | MP_ARG_REQUIRED },
79-
{ MP_QSTR_tempo, MP_ARG_INT | MP_ARG_REQUIRED },
78+
{ MP_QSTR_buffer, MP_ARG_OBJ | MP_ARG_REQUIRED, {} },
79+
{ MP_QSTR_tempo, MP_ARG_INT | MP_ARG_REQUIRED, {} },
8080
{ MP_QSTR_sample_rate, MP_ARG_INT | MP_ARG_KW_ONLY, {.u_int = 11025} },
8181
{ MP_QSTR_waveform, MP_ARG_OBJ | MP_ARG_KW_ONLY, {.u_obj = mp_const_none } },
8282
{ MP_QSTR_envelope, MP_ARG_OBJ | MP_ARG_KW_ONLY, {.u_obj = mp_const_none } },

‎shared-bindings/synthio/__init__.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ const mp_obj_namedtuple_type_t synthio_envelope_type_obj = {
233233
STATIC mp_obj_t synthio_from_file(size_t n_args, const mp_obj_t *pos_args, mp_map_t *kw_args) {
234234
enum { ARG_file, ARG_sample_rate, ARG_waveform, ARG_envelope };
235235
static const mp_arg_t allowed_args[] = {
236-
{ MP_QSTR_file, MP_ARG_OBJ | MP_ARG_REQUIRED },
236+
{ MP_QSTR_file, MP_ARG_OBJ | MP_ARG_REQUIRED, {} },
237237
{ MP_QSTR_sample_rate, MP_ARG_INT | MP_ARG_KW_ONLY, {.u_int = 11025} },
238238
{ MP_QSTR_waveform, MP_ARG_OBJ | MP_ARG_KW_ONLY, {.u_obj = mp_const_none } },
239239
{ MP_QSTR_envelope, MP_ARG_OBJ | MP_ARG_KW_ONLY, {.u_obj = mp_const_none } },

0 commit comments

Comments
 (0)