Re: collection->foreach(func)

2023-03-22 Fir de Conversatie Bram Moolenaar


[resend - message was rejected somewhere]

> * Bram Moolenaar  [230314 13:24]:
> > 
> > Ernie Rael wrote:
> > 
> > > I sometimes want "collection->foreach((_, v) => ...v...)", a simple
> > > one liner, no return or return ignored. I know it's not a performance
> > > winner, or I'd want it more.
> > > 
> > > After a quick look at the code, it looks like simply introducing
> > > another filtermap_T, FILTERMAP_FOREACH, and there's not much coding
> > > needed (I only looked at list). Probably more work doing tests/doc.
> > > 
> > > Any objections or other considerations?
> > 
> > So, this would iterate over the items in a List or Dictionary and invoke
> > a function for each one.  I assume the List or Dictionary is not
> > modified, otherwise you would use map().  Thus it's a short version of a
> > for loop.  This seems useful.
> > 
> > I would suggest first writing the help and think of any error conditions
> > that need to be handled.  Perhaps the collection should be locked
> > (frozen) to avoid trouble?
> 
> Would this have any functionality that is not provided by using mapnew
> and ignoring or throwing away the result?

It is not adding functionality, it is restricting functionality.
Using mapnew() suggests that the list/dict is going to be modified, only
by taking care it would be avoided.  And it adds overhead, a new
list/dict is created and items added to it.

-- 
FATHER: One day, lad, all this will be yours ...
PRINCE: What - the curtains?
 "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///  \\\
\\\sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/20230322164149.6867F1C07BD%40moolenaar.net.


Re: collection->foreach(func)

2023-03-14 Fir de Conversatie Ernie Rael

On 23/03/14 10:23 AM, Bram Moolenaar wrote:

Ernie Rael wrote:


I sometimes want "collection->foreach((_, v) => ...v...)", a simple
one liner, no return or return ignored. I know it's not a performance
winner, or I'd want it more.

After a quick look at the code, it looks like simply introducing
another filtermap_T, FILTERMAP_FOREACH, and there's not much coding
needed (I only looked at list). Probably more work doing tests/doc.

Any objections or other considerations?

So, this would iterate over the items in a List or Dictionary and invoke
a function for each one.  I assume the List or Dictionary is not
modified, otherwise you would use map().
Yes, yes. Not modified as though `lockvar 1`, a structured value, like 
an inner list in a list of lists, could be modified.

Thus it's a short version of a
for loop.  This seems useful.

I would suggest first writing the help and think of any error conditions
that need to be handled.

The first PR will be doc only.

   Perhaps the collection should be locked
(frozen) to avoid trouble?


To protect against the script changing the list/dict top level structure 
during iteration? Aren't map/filter/mapnew susceptible to the script 
manipulating the source collection during iteration?


But it's always nice to get an exception if the script misbehaves. In 
this case, there can be no structure modification, unlike map/filter. I 
suppose with map/filter the list could be locked during the operation 
except when the return from the function indicates a change needs to be 
made (I've never looked at the lock code; have no idea of the overhead 
of an internal `lockvar 1`)


-ernie





--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups "vim_dev" group.

To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/2b709080-3f01-a2d4-2202-56adc48208b7%40raelity.com.


Re: collection->foreach(func)

2023-03-14 Fir de Conversatie Ernie Rael

On 23/03/14 12:45 PM, Marvin Renich wrote:

* Bram Moolenaar  [230314 13:24]:

Ernie Rael wrote:


I sometimes want "collection->foreach((_, v) => ...v...)", a simple
one liner, no return or return ignored. I know it's not a performance
winner, or I'd want it more.

After a quick look at the code, it looks like simply introducing
another filtermap_T, FILTERMAP_FOREACH, and there's not much coding
needed (I only looked at list). Probably more work doing tests/doc.

Any objections or other considerations?

So, this would iterate over the items in a List or Dictionary and invoke
a function for each one.  I assume the List or Dictionary is not
modified, otherwise you would use map().  Thus it's a short version of a
for loop.  This seems useful.

I would suggest first writing the help and think of any error conditions
that need to be handled.  Perhaps the collection should be locked
(frozen) to avoid trouble?

Would this have any functionality that is not provided by using mapnew
and ignoring or throwing away the result?


All the existing functions, map/mapnew/filter, require a return value; 
that makes it impossible to have a one line lambda with a rhs that isn't 
a value. And requiring a new list to be built is painful.


-ernie


...Marvin



--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups "vim_dev" group.

To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/ce905ad3-a101-b9ec-d989-c4949a97f0d4%40raelity.com.


Re: collection->foreach(func)

2023-03-14 Fir de Conversatie Marvin Renich
* Bram Moolenaar  [230314 13:24]:
> 
> Ernie Rael wrote:
> 
> > I sometimes want "collection->foreach((_, v) => ...v...)", a simple
> > one liner, no return or return ignored. I know it's not a performance
> > winner, or I'd want it more.
> > 
> > After a quick look at the code, it looks like simply introducing
> > another filtermap_T, FILTERMAP_FOREACH, and there's not much coding
> > needed (I only looked at list). Probably more work doing tests/doc.
> > 
> > Any objections or other considerations?
> 
> So, this would iterate over the items in a List or Dictionary and invoke
> a function for each one.  I assume the List or Dictionary is not
> modified, otherwise you would use map().  Thus it's a short version of a
> for loop.  This seems useful.
> 
> I would suggest first writing the help and think of any error conditions
> that need to be handled.  Perhaps the collection should be locked
> (frozen) to avoid trouble?

Would this have any functionality that is not provided by using mapnew
and ignoring or throwing away the result?

...Marvin

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/ZBDOze8DACtOID%2Bj%40basil.wdw.


Re: collection->foreach(func)

2023-03-14 Fir de Conversatie Bram Moolenaar


Ernie Rael wrote:

> I sometimes want "collection->foreach((_, v) => ...v...)", a simple
> one liner, no return or return ignored. I know it's not a performance
> winner, or I'd want it more.
> 
> After a quick look at the code, it looks like simply introducing
> another filtermap_T, FILTERMAP_FOREACH, and there's not much coding
> needed (I only looked at list). Probably more work doing tests/doc.
> 
> Any objections or other considerations?

So, this would iterate over the items in a List or Dictionary and invoke
a function for each one.  I assume the List or Dictionary is not
modified, otherwise you would use map().  Thus it's a short version of a
for loop.  This seems useful.

I would suggest first writing the help and think of any error conditions
that need to be handled.  Perhaps the collection should be locked
(frozen) to avoid trouble?

-- 
This sentence is not sure that it exists, but if it does, it will
certainly consider the possibility that other sentences exist.

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///  \\\
\\\sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/20230314172354.B208F1C0CA2%40moolenaar.net.


collection->foreach(func)

2023-03-13 Fir de Conversatie Ernie Rael

I sometimes want "collection->foreach((_, v) => ...v...)", a simple
one liner, no return or return ignored. I know it's not a performance
winner, or I'd want it more.

After a quick look at the code, it looks like simply introducing
another filtermap_T, FILTERMAP_FOREACH, and there's not much coding
needed (I only looked at list). Probably more work doing tests/doc.

Any objections or other considerations?

-ernie

--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups "vim_dev" group.

To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/b8e2b8fa-c6d8-cc41-ac63-6b1d21a75f41%40raelity.com.