Tíquete #45720

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

: 2022-09-26 05:36 Última Atualização: 2022-10-06 18:12

Relator:
Dono:
Tipo:
Estado:
Fechado
Componente:
Marcos:
Prioridade:
5 - Medium
Gravidade:
5 - Medium
Resolução:
Fixed
Arquivo:
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:41 Updated by: cazfi
  • Dono Update from (Nenhum) to cazfi
  • Resolução Update from Nenhum to Accepted
  • Marco Update from 3.0.5 (fechado) to 3.2.0
Comentário

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
  • Resolução Update from Accepted to Nenhum
Comentário

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

2022-10-05 10:05 Updated by: cazfi
  • Resolução Update from Nenhum to Accepted
Comentário

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
  • Estado Update from Aberto to Fechado
  • Resolução Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Login