Opened 12 years ago

Last modified 15 months ago

#4929 assigned enhancement

Rename dprintf() to Avoid Clash with the POSIX.1-2008 Function

Reported by: bonefish Owned by: nobody
Priority: normal Milestone: R1.1
Component: System/Kernel Version: R1/alpha1
Keywords: Cc:
Blocked By: Blocking: #15366
Platform: All

Description

POSIX.1-2008 introduces the functions [v]dprintf(). To avoid a name clash when we implement them, the kernel dprintf() should be renamed to debug_printf(). We can continue provide the old symbol for binary compatibility.

Change History (10)

comment:1 by waddlesplash, 6 years ago

Milestone: R1Unscheduled

comment:2 by waddlesplash, 6 years ago

Milestone: UnscheduledR1

Reverting earlier milestone change

comment:3 by axeld, 5 years ago

Owner: changed from axeld to nobody
Status: newassigned

comment:4 by waddlesplash, 3 years ago

Why rename it to "debug_printf" instead of just "printf", anyway?

comment:5 by waddlesplash, 2 years ago

Blocking: 15366 added

comment:6 by X512, 17 months ago

Is it really needed? Kernel is not expected to provide full POSIX support.

comment:7 by waddlesplash, 17 months ago

The problem also applies to libroot.

in reply to:  7 comment:8 by mmlr, 17 months ago

Replying to waddlesplash:

The problem also applies to libroot.

No, in userland it is already called debug_printf and libroot exports the other dprintf(fd, format, ...) from stdio.

Last edited 17 months ago by mmlr (previous) (diff)

comment:9 by pulkomandy, 17 months ago

It's not a "needed" change, but just to avoid confusion by having two different functions in two different places doing different things.

The kernel is of course not implementing the full POSIX API, but we at least try to keep it as close as reasonably possible as it makes development a lot easier and removes a lot of "oh no I'm writing kernel code, I need magic incantations" feeling by having a subset of the same APIs available.

Last edited 17 months ago by pulkomandy (previous) (diff)

comment:10 by pulkomandy, 15 months ago

Milestone: R1R1.1
Note: See TracTickets for help on using tickets.