[nerobotParticipant]

Next and Wend are more readable for me.

[nerobotParticipant]

Probably, it’s a lang bug. The code looks correct.

[nerobotParticipant]

Nice work!

There will be good to have “Read more” button in News section – to see short/clean blog entries and jump into if needed.

[nerobotParticipant]

You cat to simplify a bit logic with iterator:

Monkey

' current variant If pb.model.Y < -20 pb.model.Scale = pb.model.Scale * 0.75 If pb.model.Scale.x < 0.01 pb.model.Destroy () iter.Erase () Else iter.Bump () Endif Else iter.Bump () Endif ' new variant If pb.model.Y < -20 pb.model.Scale = pb.model.Scale * 0.75 If pb.model.Scale.x < 0.01 pb.model.Destroy () iter.Erase () Continue Endif Endif iter.Bump ()

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

' current variant If pb.model.Y < -20                                   pb.model.Scale = pb.model.Scale * 0.75                                   If pb.model.Scale.x < 0.01                                         pb.model.Destroy ()                                          iter.Erase ()                                  Else                                          iter.Bump ()                                 Endif                           Else                                 iter.Bump ()                         Endif   ' new variant If pb.model.Y < -20     pb.model.Scale = pb.model.Scale * 0.75     If pb.model.Scale.x < 0.01     pb.model.Destroy ()     iter.Erase ()     Continue   Endif   Endif   iter.Bump ()

[nerobotParticipant]

Yes, it’s time to add setup wizard.

I like prime themes except of bold font. That’s why I revert default theme to warm.

If wizard will take a long time I can set back prime one.

[nerobotParticipant]

It’s in progress.

[nerobotParticipant]

Very nice explanation to be article-ized.

[nerobotParticipant]

Are you afraid of iterators?

My example above is an analogue of native Eachin.

I think it’s the fastest and easiest way.

[nerobotParticipant]

Try to remove items inside of ‘foreach’ in Java or C# and you’ll get runtime error “collection was modified due iterating” (text isn’t exactly).

To be able to remove items due iterating you have to use stack.iterator with while loop:

Monkey

Local iter:=stack.All () While Not iter.AtEnd ()   Local value:=iter.Current   If value=needRemove     iter.Erase ()   Else     iter.Bump ()   Endif Wend

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

Local iter:=stack.All ()   While Not iter.AtEnd ()     Local value:=iter.Current     If value=needRemove       iter.Erase ()     Else       iter.Bump ()     Endif   Wend

[nerobotParticipant]

Tabs to spaces – will be added.

Docs manuals – will be added. I don’t know why you can’t drag divider, it should to work.

[nerobotParticipant]

Thanks.

Classes, methods etc have start and end positions, but parser knows nothing about inner code elements like If-s this time, it’s not difficult to teach it but multilined stuff is a problem. Maybe Mark help me when I will implement that.

Folding is in todo.

What does “indent by fold feature” mean?

[nerobotParticipant]

@cocon

/parser/Monkey2Parser.m2

/view/AutocompletionDialog.m2 (there are few related classes)

/view/ListViewExt.m2 – my custom listview that renders items

/document/CodeDocument.m2

* .m2 – to be shorter

[nerobotParticipant]

About FPS – it depends on SwapInterval, is there waiting for vsync or not.

If rendertime is small then waiting time will be longer to make fps count const.

[nerobotParticipant]

It worked for me. But this code is technically equivalent to above one. So it can produce null pointer.

As a solution for prefs, we can initialize _ptefs field right inside of Get () if it is null.

Consts initialization is a weak side of monkey2, that may cause errors, so that it’s important to have related remarks in docs.

(Waiting for a developer that can bring dependency checking.)

[nerobotParticipant]

Look at this

https://github.com/blitz-research/monkey2/blob/34825f8b197c9dfa721888d21193ba93ac0b5728/modules/mojo/app/app.monkey2#L593

There is a WHILE loop that grab all events. Maybe we need to grab by portions or every second frame or … ?

I didn’t test how much millisecs are needed to dispatch holded keys. And maybe the bottleneck isn’t here.

[Author]

Posts