Typo in ground/1 call?

[pmoura]

In the following clause:

packages-pengines/pengines_io.pl

Lines 651 to 658 in 463c373

	map_output(ID, Term, json{event:output, id:ID, data:Data}) :-
	( atomic(Term)
	-> Data = Term
	; is_dict(Term, json),
	ground(json) % TBD: Check proper JSON object?
	-> Data = Term
	; term_string(Term, Data)
	).

Do you meant to write instead ground(Term)?

[JanWielemaker]

That is most likely what was intended. I think the remark also suggests that a ground test is
not ok though, so I think I'll leave it as is for now. Nice catch of dubious code though!

[pmoura]

The Logtalk linter flags the ground(json) goal as a goal that's always true. This lint check can be fouled, however. It's also flagging elsewhere in the SWI-Prolog library the goal random(2) =:= 0 was always being false 🙃