kc3-lang/harfbuzz

2024-07-08T15:33:39

[subset] Make sure the clamp is done in a int64_t space Otherwise nags about things like this, In member function ‘int64_t graph::graph_t::vertex_t::modified_distance(unsigned int) const’, inlined from ‘void graph::graph_t::sort_shortest_distance()’ at ../src/graph/graph.hh:626:24: ../src/graph/graph.hh:371:20: warning: dangling pointer to an unnamed temporary may be used [-Wdangling-pointer=] 371 | hb_clamp (distance + distance_modifier (), (uint64_t) 0, (uint64_t) 0x7FFFFFFFFFF); | ~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ And some of the CI bots fail also like this https://github.com/harfbuzz/harfbuzz/actions/runs/9838686960/job/27159310858?pr=4793 But it probably something else also can be improved which maybe is out of scope for this particular change.