Skip to content

Commit

Permalink
Merge pull request #67 from gap-packages/avoid
Browse files Browse the repository at this point in the history
avoid trivial function wrappers with ForAll
  • Loading branch information
cdwensley authored Jan 8, 2024
2 parents f082355 + 221808e commit 106a5ce
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 6 deletions.
3 changes: 3 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
# CHANGES log for the 'Utils' package

## Version 0.84dev for GAP 4.12.2 (08/01/24)
* (08/01/24) avoid tgrivial function wrappers in List and ForAll

## Version 0.84 for GAP 4.12.2 (11/09/23)
* (11/09/23) changed manual and test for DirectSumDecompositionMatrices

Expand Down
4 changes: 2 additions & 2 deletions PackageInfo.g
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ SetPackageInfo( rec(

PackageName := "utils",
Subtitle := "Utility functions in GAP",
Version := "0.84",
Date := "11/09/2023", # dd/mm/yyyy format
Version := "0.84dev",
Date := "08/01/2024", # dd/mm/yyyy format
License := "GPL-2.0-or-later",

Persons := [
Expand Down
8 changes: 4 additions & 4 deletions lib/print.gi
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
##
#W print.gi GAP4 package `Utils' Chris Wensley
##
#Y Copyright (C) 2015-2018, The GAP Group
#Y Copyright (C) 2015-2024, The GAP Group

#############################################################################
## these operations have not been transferred from any other package
Expand All @@ -24,7 +24,7 @@ function( arg )
elif ( nargs = 2 ) then
L := arg[2];
if IsList( L ) then
ok := ForAll( L, n -> IsPosInt(n) );
ok := ForAll( L, IsPosInt );
if not ok then
Error( "L must only contain positive integers" );
fi;
Expand Down Expand Up @@ -120,7 +120,7 @@ function( L, posL )

local len, i;

if not ForAll( posL, n -> IsPosInt( n ) ) then
if not ForAll( posL, IsPosInt ) then
Error( "posL should be a list of positive integers" );
fi;
len := Length( L );
Expand All @@ -137,7 +137,7 @@ function( iter0, posL )

local iter, max, i, x;

if not ForAll( posL, n -> IsPosInt( n ) ) then
if not ForAll( posL, IsPosInt ) then
Error( "posL should be a list of positive integers" );
fi;
iter := ShallowCopy( iter0 );
Expand Down

0 comments on commit 106a5ce

Please sign in to comment.