RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-27 19:47:34 +00:00
Code Issues Activity Actions

LibPDF: Require Document* in Parser constructor

Browse source 
This makes it a bit easier to avoid calling parser->set_document, an
issue which cost me ~30 minutes to find.
This commit is contained in:
Matthew Olsson 2022-03-22 19:22:45 -07:00 • committed by Andreas Kling
parent a8de9cf541
commit 60c3e786be
3 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibPDF/Parser.cpp
View file

@ -22,14 +22,15 @@ static NonnullRefPtr<T> make_object(Args... args) requires(IsBaseOf<Object, T>)
return adopt_ref(*new T(forward<Args>(args)...));
}
PDFErrorOr<Vector<Command>> Parser::parse_graphics_commands(ReadonlyBytes bytes)
PDFErrorOr<Vector<Command>> Parser::parse_graphics_commands(Document* document, ReadonlyBytes bytes)
{
auto parser = adopt_ref(*new Parser(bytes));
auto parser = adopt_ref(*new Parser(document, bytes));
return parser->parse_graphics_commands();
}
Parser::Parser(Badge<Document>, ReadonlyBytes bytes)
Parser::Parser(Document* document, ReadonlyBytes bytes)
: m_reader(bytes)
, m_document(document)
{
}

4
Userland/Libraries/LibPDF/Parser.h
View file

@ -25,9 +25,9 @@ public:
Linearized,
};
static PDFErrorOr<Vector<Command>> parse_graphics_commands(ReadonlyBytes);
static PDFErrorOr<Vector<Command>> parse_graphics_commands(Document*, ReadonlyBytes);
Parser(Badge<Document>, ReadonlyBytes);
Parser(Document*, ReadonlyBytes);
[[nodiscard]] ALWAYS_INLINE RefPtr<DictObject> const& trailer() const { return m_trailer; }
void set_document(WeakPtr<Document> const&);

2
Userland/Libraries/LibPDF/Renderer.cpp
View file

@ -77,7 +77,7 @@ PDFErrorOr<void> Renderer::render()
byte_buffer.append(bytes.data(), bytes.size());
}
auto commands = TRY(Parser::parse_graphics_commands(byte_buffer));
auto commands = TRY(Parser::parse_graphics_commands(m_document, byte_buffer));
for (auto& command : commands)
TRY(handle_command(command));