Skip to content
This repository has been archived by the owner on Jan 23, 2022. It is now read-only.

Added a fix to RedoDims that allows actually setting the $SIZE macro. #4

Closed
wants to merge 1 commit into from

Conversation

mohawk2
Copy link
Member

@mohawk2 mohawk2 commented Aug 30, 2015

@drzowie says:

This hasn't worked in, well, ever due to a scope bug in the macro definition. (never nest closures if you can help it...)

This hasn't worked in, well, ever due to a scope bug in the macro
definition.  (never nest closures if you can help it...)
croak "can't get SIZE of undefined dimension (RedoDims=$redodims)."
unless defined $dimobjs->{$redodims};
return $dimobjs->{$redodims}->get_size();
eval 'use PDL::IO::Dumper';
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mohawk2 mohawk2 mentioned this pull request Aug 30, 2015
@mohawk2 mohawk2 closed this May 15, 2019
@mohawk2 mohawk2 deleted the RedoDimsCode-fix branch May 15, 2019 14:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant