0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

LibEDID: Exclude DMT::MonitorTiming::name() from the Kernel

Browse source 
This API is only used by userland and it uses infallible Strings, so
let's just ifdef it out of the Kernel.
This commit is contained in:
Idan Horowitz 2022-02-15 23:30:00 +02:00 • committed by Andreas Kling
parent 3219ce3d61
commit 410183a7b0
Notes: sideshowbarker 2024-07-17 18:40:25 +09:00
2 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibEDID/DMT.cpp
View file

@ -4,9 +4,12 @@
* SPDX-License-Identifier: BSD-2-Clause
*/
#include <AK/String.h>
#include <LibEDID/DMT.h>
#ifndef KERNEL
# include <AK/String.h>
#endif
namespace EDID {
// Monitor timings as per Display Monitor Timing Standard (DMT) 1.0 rev 13
@ -116,12 +119,14 @@ u32 DMT::MonitorTiming::refresh_rate_hz() const
return vertical_frequency_hz().ltrunk();
}
#ifndef KERNEL
String DMT::MonitorTiming::name() const
{
if (scan_type == ScanType::Interlaced)
return String::formatted("{} x {} @ {}Hz (Interlaced)", horizontal_pixels, vertical_lines, refresh_rate_hz());
return String::formatted("{} x {} @ {}Hz", horizontal_pixels, vertical_lines, refresh_rate_hz());
}
#endif
auto DMT::find_timing_by_dmt_id(u8 dmt_id) -> MonitorTiming const*
{

2
Userland/Libraries/LibEDID/DMT.h
View file

@ -53,7 +53,9 @@ public:
FixedPoint<16, u32> horizontal_frequency_khz() const;
FixedPoint<16, u32> vertical_frequency_hz() const;
u32 refresh_rate_hz() const;
#ifndef KERNEL
String name() const;
#endif
};
static MonitorTiming const* find_timing_by_dmt_id(u8);