Replace `.Call()` expressions with `_impl` functions and add explanatory comments#2562

Draft

Copilot wants to merge 21 commits intomainfrom

copilot/replace-call-expressions-with-impl-functions

Commits on Jan 27, 2026

Initial plan
Copilot
committed
Add comments explaining why .Call() expressions cannot be replaced with _impl functions

Copilot
and
krlmlr
committed
Fix syntax error in attributes.R comment placement

Copilot
and
krlmlr
committed
Replace .Call() with _impl functions where possible, add explanatory comments elsewhere

Copilot
and
krlmlr
committed
Replace subcomponent, girth, ego_size, ego, make_ego_graph with _impl functions

Copilot
and
krlmlr
committed
Add comments to remaining .Call() expressions in structural-properties.R

Copilot
and
krlmlr
committed
Add comment explaining efficiency trade-off in girth() using girth_impl()

Copilot
and
krlmlr
committed
Update BFS/DFS comments to reference bfs_closure_impl/dfs_closure_impl

Copilot
and
krlmlr
committed
Add comments to internal .Call() expressions in attributes.R

Copilot
and
krlmlr
committed
Add comments and replacements for .Call() in bipartite.R, centrality.R, cliques.R, community.R

Copilot
and
krlmlr
committed
Add comments to all remaining .Call() expressions across R files

Copilot
and
krlmlr
committed
Replace .Call() with _impl functions in layout, interface, foreign files; fix API adaptations

Copilot
and
krlmlr
committed
Fix layout functions to properly initialize coords when NULL before calling _impl

Copilot
and
krlmlr
committed
Fix weight handling in layout functions to let _impl handle defaults

Copilot
and
krlmlr
committed
Revert some foreign.R functions to .Call() due to _impl bugs or different semantics

Copilot
and
krlmlr
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Replace `.Call()` expressions with `_impl` functions and add explanatory comments#2562

Replace `.Call()` expressions with `_impl` functions and add explanatory comments#2562
Copilot wants to merge 21 commits intomainfrom
copilot/replace-call-expressions-with-impl-functions

Commits on Jan 27, 2026

Initial plan

Add comments explaining why .Call() expressions cannot be replaced with _impl functions

Fix syntax error in attributes.R comment placement

Replace .Call() with _impl functions where possible, add explanatory comments elsewhere

Replace subcomponent, girth, ego_size, ego, make_ego_graph with _impl functions

Add comments to remaining .Call() expressions in structural-properties.R

Add comment explaining efficiency trade-off in girth() using girth_impl()

Update BFS/DFS comments to reference bfs_closure_impl/dfs_closure_impl

Add comments to internal .Call() expressions in attributes.R

Add comments and replacements for .Call() in bipartite.R, centrality.R, cliques.R, community.R

Add comments to all remaining .Call() expressions across R files

Replace .Call() with _impl functions in layout, interface, foreign files; fix API adaptations

Fix layout functions to properly initialize coords when NULL before calling _impl

Fix weight handling in layout functions to let _impl handle defaults

Revert some foreign.R functions to .Call() due to _impl bugs or different semantics

Commits on Jan 30, 2026

Replace more .Call() with _impl functions: count_components, count_max_cliques, bipartite_projection, disjoint_union

Add igraph API comments to remaining .Call() expressions

Remove errant snapshot file

Add detailed igraph API function names to remaining .Call() comments

Fix type comparison bug in bipartite_projection warning

Add igraph API function names to .Call() comments; replace spinglass with _impl