Improvement: VA Assist Pro / Search Methods including

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

Improvement: VA Assist Pro / Search Methods including

jtuchel
I would like to suggest an improvement for VA Assist Pro.

If you search for Methods including a String, the whole process takes a lot of time. That is nothing new.
But the search blocks the whole image and unfortunately also doesn't give you a chance to look at the first hits it found thus far. Maybe the first one is already my candidate and I could cancel the search...

So instead of this:




Some browser that shows me those hits so far and the search in a background process might make the exoerience so much better. It should be possible to end the search and it should be possible to still browser around - even if this may have an impact on the search results...

What do people think?

Joachim

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

Louis LaBrunda
Hi Joachim,

+1 and how about a case insensitive option.

Lou

On Tuesday, May 30, 2017 at 5:13:20 AM UTC-4, Joachim Tuchel wrote:
I would like to suggest an improvement for VA Assist Pro.

If you search for Methods including a String, the whole process takes a lot of time. That is nothing new.
But the search blocks the whole image and unfortunately also doesn't give you a chance to look at the first hits it found thus far. Maybe the first one is already my candidate and I could cancel the search...

So instead of this:




Some browser that shows me those hits so far and the search in a background process might make the exoerience so much better. It should be possible to end the search and it should be possible to still browser around - even if this may have an impact on the search results...

What do people think?

Joachim

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

Klaus Breker-2
Hi,

we are using this code (example) to search case insensitive for strings in methods:

    | anUppercaseString aCompiledMethodCollection |

    anUppercaseString := 'Smalltalk'  asUppercase.
    aCompiledMethodCollection := CompiledMethod allInstances.

    (StsHighlightingMethodsBrowser
        on: (aCompiledMethodCollection select: [:cm | cm includesSubstring: uppercaseString]) asSet
        labeled: aString
        highlighting: aString)
            open

It is quite fast.

Regards

Klaus

Am Dienstag, 30. Mai 2017 15:13:43 UTC+2 schrieb Louis LaBrunda:
Hi Joachim,

+1 and how about a case insensitive option.

Lou

On Tuesday, May 30, 2017 at 5:13:20 AM UTC-4, Joachim Tuchel wrote:
I would like to suggest an improvement for VA Assist Pro.

If you search for Methods including a String, the whole process takes a lot of time. That is nothing new.
But the search blocks the whole image and unfortunately also doesn't give you a chance to look at the first hits it found thus far. Maybe the first one is already my candidate and I could cancel the search...

So instead of this:




Some browser that shows me those hits so far and the search in a background process might make the exoerience so much better. It should be possible to end the search and it should be possible to still browser around - even if this may have an impact on the search results...

What do people think?

Joachim

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

Lawrence Trutter
In reply to this post by Louis LaBrunda
+1

Larry 

On Tuesday, May 30, 2017 at 8:13:43 AM UTC-5, Louis LaBrunda wrote:
Hi Joachim,

+1 and how about a case insensitive option.

Lou

On Tuesday, May 30, 2017 at 5:13:20 AM UTC-4, Joachim Tuchel wrote:
I would like to suggest an improvement for VA Assist Pro.

If you search for Methods including a String, the whole process takes a lot of time. That is nothing new.
But the search blocks the whole image and unfortunately also doesn't give you a chance to look at the first hits it found thus far. Maybe the first one is already my candidate and I could cancel the search...

So instead of this:




Some browser that shows me those hits so far and the search in a background process might make the exoerience so much better. It should be possible to end the search and it should be possible to still browser around - even if this may have an impact on the search results...

What do people think?

Joachim

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

Santiago Cardoso Geller-3
In reply to this post by Louis LaBrunda
We use the following:



buscarString: aString
        "Devuelve los métodos que hacen referencia a aString."

    ^(CompiledMethod allInstances select: [ :each |
        (each stringLiterals select: [ :eachString |
            eachString asUppercase abtContains: aString asUppercase]) notEmpty ]) inspect




2017-05-30 10:13 GMT-03:00 Louis LaBrunda <[hidden email]>:
Hi Joachim,

+1 and how about a case insensitive option.

Lou


On Tuesday, May 30, 2017 at 5:13:20 AM UTC-4, Joachim Tuchel wrote:
I would like to suggest an improvement for VA Assist Pro.

If you search for Methods including a String, the whole process takes a lot of time. That is nothing new.
But the search blocks the whole image and unfortunately also doesn't give you a chance to look at the first hits it found thus far. Maybe the first one is already my candidate and I could cancel the search...

So instead of this:




Some browser that shows me those hits so far and the search in a background process might make the exoerience so much better. It should be possible to end the search and it should be possible to still browser around - even if this may have an impact on the search results...

What do people think?

Joachim

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

Santiago Cardoso Geller-3
Sorry, this method implementation is required:

CompiledMethod>>#stringLiterals
^self allLiterals select: [ :eachLiteral | eachLiteral class == String ]

2017-05-30 15:50 GMT-03:00 Santiago Cardoso Geller <[hidden email]>:
We use the following:



buscarString: aString
        "Devuelve los métodos que hacen referencia a aString."

    ^(CompiledMethod allInstances select: [ :each |
        (each stringLiterals select: [ :eachString |
            eachString asUppercase abtContains: aString asUppercase]) notEmpty ]) inspect




2017-05-30 10:13 GMT-03:00 Louis LaBrunda <[hidden email]>:
Hi Joachim,

+1 and how about a case insensitive option.

Lou


On Tuesday, May 30, 2017 at 5:13:20 AM UTC-4, Joachim Tuchel wrote:
I would like to suggest an improvement for VA Assist Pro.

If you search for Methods including a String, the whole process takes a lot of time. That is nothing new.
But the search blocks the whole image and unfortunately also doesn't give you a chance to look at the first hits it found thus far. Maybe the first one is already my candidate and I could cancel the search...

So instead of this:




Some browser that shows me those hits so far and the search in a background process might make the exoerience so much better. It should be possible to end the search and it should be possible to still browser around - even if this may have an impact on the search results...

What do people think?

Joachim

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.


--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

Ben van Dijk
In reply to this post by jtuchel
+1

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: Improvement: VA Assist Pro / Search Methods including

dmacq
In reply to this post by jtuchel
Thanks to all of you for your suggestions. And the code!

I will add this to Instantiations to do list. The major issue for us is ensuring that a change like this does not break anything else.

Donald [|]

--
You received this message because you are subscribed to the Google Groups "VA Smalltalk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
Visit this group at https://groups.google.com/group/va-smalltalk.
For more options, visit https://groups.google.com/d/optout.