Skip to content

Commit 3c8ab23

Browse files
npigginelmarco
authored andcommitted
chardev: Remove __-prefixed names
Peter points out double underscore prefix names tend to be reserved for the system. Clean these up. Suggested-by: Peter Maydell <[email protected]> Signed-off-by: Nicholas Piggin <[email protected]> Reviewed-by: Marc-André Lureau <[email protected]> Reviewed-by: Peter Maydell <[email protected]> Message-ID: <[email protected]>
1 parent 651b386 commit 3c8ab23

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

chardev/char.c

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -633,8 +633,8 @@ static void qemu_chardev_set_replay(Chardev *chr, Error **errp)
633633
}
634634
}
635635

636-
static Chardev *__qemu_chr_new_from_opts(QemuOpts *opts, GMainContext *context,
637-
bool replay, Error **errp)
636+
static Chardev *do_qemu_chr_new_from_opts(QemuOpts *opts, GMainContext *context,
637+
bool replay, Error **errp)
638638
{
639639
const ChardevClass *cc;
640640
Chardev *base = NULL, *chr = NULL;
@@ -712,12 +712,12 @@ Chardev *qemu_chr_new_from_opts(QemuOpts *opts, GMainContext *context,
712712
Error **errp)
713713
{
714714
/* XXX: should this really not record/replay? */
715-
return __qemu_chr_new_from_opts(opts, context, false, errp);
715+
return do_qemu_chr_new_from_opts(opts, context, false, errp);
716716
}
717717

718-
static Chardev *__qemu_chr_new(const char *label, const char *filename,
719-
bool permit_mux_mon, GMainContext *context,
720-
bool replay)
718+
static Chardev *qemu_chr_new_from_name(const char *label, const char *filename,
719+
bool permit_mux_mon,
720+
GMainContext *context, bool replay)
721721
{
722722
const char *p;
723723
Chardev *chr;
@@ -740,7 +740,7 @@ static Chardev *__qemu_chr_new(const char *label, const char *filename,
740740
if (!opts)
741741
return NULL;
742742

743-
chr = __qemu_chr_new_from_opts(opts, context, replay, &err);
743+
chr = do_qemu_chr_new_from_opts(opts, context, replay, &err);
744744
if (!chr) {
745745
error_report_err(err);
746746
goto out;
@@ -765,15 +765,17 @@ static Chardev *__qemu_chr_new(const char *label, const char *filename,
765765
Chardev *qemu_chr_new_noreplay(const char *label, const char *filename,
766766
bool permit_mux_mon, GMainContext *context)
767767
{
768-
return __qemu_chr_new(label, filename, permit_mux_mon, context, false);
768+
return qemu_chr_new_from_name(label, filename, permit_mux_mon, context,
769+
false);
769770
}
770771

771772
static Chardev *qemu_chr_new_permit_mux_mon(const char *label,
772773
const char *filename,
773774
bool permit_mux_mon,
774775
GMainContext *context)
775776
{
776-
return __qemu_chr_new(label, filename, permit_mux_mon, context, true);
777+
return qemu_chr_new_from_name(label, filename, permit_mux_mon, context,
778+
true);
777779
}
778780

779781
Chardev *qemu_chr_new(const char *label, const char *filename,

0 commit comments

Comments
 (0)