requirements.h: replace "astring.h" include with a forward declaration

• Reporter: cazfi
• Owner: cazfi
• Type: Patches
• Status: Closed
• Component: General
• MileStone: 3.2.0
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: 1

Details

When I ported #45544, I realized that "astring.h" include isn't needed in requirements.h. Rework that also in the newer branches.

Ticket History (3/7 Histories)

2022-09-26 05:36 Updated by: cazfi

• New Ticket "rquirements.h: replace "astring.h" include with a forward declaration" created

2022-10-02 23:39 Updated by: cazfi

• File 0043-Remove-unnecessary-includes-from-common-headers.patch (File ID: 10447) is attached

2022-10-02 23:41 Updated by: cazfi

• Owner Update from (None) to cazfi
• Resolution Update from None to Accepted
• Milestone Update from 3.0.5 (closed) to 3.2.0

Comment

Went through ALL the headers in common/ to remove unnecessary includes. For master. The resulting patch would be too much work to port to earlier branches, for the minimal benefit it provides.

2022-10-03 11:52 Updated by: cazfi

• Summary Updated

2022-10-05 09:48 Updated by: cazfi

• Resolution Update from Accepted to None

Comment

Likely this patch is the one currently breaking tex build on meson.

2022-10-05 10:05 Updated by: cazfi

• Resolution Update from None to Accepted

Comment

Reply To cazfi

Likely this patch is the one currently breaking tex build on meson.

No, this is ok. I just have a local patch that needs to be updated to work on top of this.

2022-10-06 18:12 Updated by: cazfi

• Status Update from Open to Closed
• Resolution Update from Accepted to Fixed