Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • May 14 04:41

    AArnott on v16.10

    Insert into 16.10 branch (compare)

  • May 14 04:41

    AArnott on main

    Insert into 16.10 branch Merge branch 'v16.10' (compare)

  • May 12 14:21

    AArnott on v16.10

    Fix OptProf for slow moving bra… Merge pull request #848 from AA… Merge remote-tracking branch 'u… (compare)

  • May 12 14:21

    AArnott on main

    Fix OptProf for slow moving bra… Merge pull request #848 from AA… Merge remote-tracking branch 'u… and 1 more (compare)

  • May 12 14:20

    AArnott on v16.9

    Fix OptProf for slow moving bra… Merge pull request #848 from AA… (compare)

  • May 12 14:20
    AArnott closed #848
  • May 12 14:02
    AArnott milestoned #848
  • May 12 14:02
    AArnott assigned #848
  • May 12 14:02
    AArnott opened #848
  • May 11 19:21

    AArnott on main

    Update insertion branch (compare)

  • May 11 18:06

    lifengl on lifengl

    (compare)

  • May 11 18:06

    lifengl on main

    Use strong references for JTF d… Merge pull request #847 from mi… (compare)

  • May 11 18:06
    lifengl closed #847
  • May 11 18:05
    lifengl commented #847
  • May 10 17:55
    nbarbettini commented #584
  • May 10 15:51
    springy76 commented #584
  • May 08 19:58
    AArnott commented #584
  • May 07 20:24
    lifengl edited #847
  • May 07 20:22
    lifengl opened #847
  • May 07 07:44
    Bosch-Eli-Black closed #827
Sam Harwell
@sharwell
so the data is sent to codecov.io but then dropped on their end
Andrew Arnott
@AArnott
I'm having a hard time finding good docs.
For example, what exactly is the pattern matching supported in the -excludeByFile switch
and what is the codecov yml file format?
or rather, which settings are supported
Sam Harwell
@sharwell
There's yml format?
Andrew Arnott
@AArnott
yes
also there seems to be a stability problem. Besides making each build 4 minutes instead of 2 (which I can live with), the latest build is 11+ minutes and growing. I suspect it's actually hung.
Sam Harwell
@sharwell
there's a possibility that opencover somehow gets strong references to objects you think should be collectible
I've seen code coverage tooling change results for GC-related tests
I normally run GC tests separately from coverage
Andrew Arnott
@AArnott
Thanks. That's pretty limited. I was hoping it could take the ExcludeByAttribute switch to the yml as well.
Any idea where the doc is for what this means: -filter:"+[Microsoft.VisualStudio.Threading*]*"?
Andrew Arnott
@AArnott
Sam Harwell
@sharwell
It appears there is a race condition in AsyncAutoResetEvent.Set, where it's possible for a task to be cancelled between when Set calls trySetResultToDefault and when the result is actually set.
Sam Harwell
@sharwell
Ah, I see, you protect in from cancellation in that case by essentially unlinking the cancellation token. Interesting.
Andrew Arnott
@AArnott
it's been a long time since I was in the zone for that code. So I'm glad you are satisfied or it would take me a while of study to figure it out again. :)
Sam Harwell
@sharwell
@AArnott :bug: Microsoft/vs-threading#75
Andrew Arnott
@AArnott
Nice catch.
Sam Harwell
@sharwell
@AArnott I think AsyncQueueTests.NoLockHeldForCancellationContinuation would benefit from an assertion at the end that the queue contains one item.
Andrew Arnott
@AArnott
Thanks. Please feel free to file issues for these suggestions (either an individual issues or one with a checklist of your suggestions) so I don't lose track of them in this stream of messages.
Sam Harwell
@sharwell
NP wasn't sure if you wanted small issues like that but they are easy to file
Andrew Arnott
@AArnott
Well, I normally wouldn't. But you tend to have a good level of insight. And ya, if you expect lots of small ideas, a single issue with a markdown checkbox list would probably be preferable.
Sam Harwell
@sharwell
Normally I would just submit pull requests but I'm mostly reviewing the code in the context of different project so issues it is for now :smile:
Sam Harwell
@sharwell
@AArnott Do you have more information about when .NET inlines continuations that don't have ExecuteSynchronously set explicitly?
Andrew Arnott
@AArnott
@sharwell don't submit PRs at this point. Our CONTRIB doc mentions this. We can't accept external PRs at the moment. However if interest is significant enough (from folks like yourself) we will prioritize getting that changed.
Sam Harwell
@sharwell
I did see that but thanks for the heads-up
Andrew Arnott
@AArnott
I'm not sure. I just know that that flag is a suggestion, but TPL may or may not inline the continuation in spite of its presence or absence.
Sam Harwell
@sharwell
PRs also serve to better demonstrate some of the items I've mentioned even if they can't be accepted directly
Andrew Arnott
@AArnott
It certainly can help encourage inlining. I'm not against the flag. I'm just pointing out that it may not actually win us anything if inlining already tends to happen.
re: PRs: True. But then we might get into this uncomfortable world of "I'm rejecting your PR, but I'm going to copy your code into my own commits". So I suppose if you want to show code, use a gist and keep it gist-y. Don't tempt me to reuse your code. :)
Sam Harwell
@sharwell
I'm like :expressionless: each place I read Till which is only 1 character shorter than Until
Andrew Arnott
@AArnott
Are you referring to test names?
Oh, perhaps JoinTillEmpty?
Sam Harwell
@sharwell
That's the one!
Andrew Arnott
@AArnott
That's feedback I would have been happy to respond to before the API was released. Too bad we didn't originally develop this in an OSS way, eh?
Sam Harwell
@sharwell
:question: Does JoinableTaskFactory [need to] propagate the ExecutionContext? I notice that it doesn't implement ICriticalNotifyCompletion, but it also seems to assume the caller is responsible for propagating the execution context.
Andrew Arnott
@AArnott
@sharwell it sounds like your question is about the type returned from SwitchToMainThreadAsync, is that right? If so, no, custom awaiters don't have to worry about ExecutionContext propagation. The runtime/BCL takes care of that for us.
Sam Harwell
@sharwell
:+1:
Sam Harwell
@sharwell
I'm having trouble running the tests inside Visual Studio. Is there something special I need to do other than what's listed in the contributing file for building?
------ Discover test started ------
[xUnit.net 00:00:00.3651571] Skipping: Microsoft.VisualStudio.Threading.Analyzers.Tests (could not find dependent assembly 'Microsoft.VisualStudio.Threading.Analyzers.Tests, Version=15.0.0')
========== Discover test finished: 0 found (0:00:00.4765306) ==========
Andrew Arnott
@AArnott
try launching VS from a Developer Command Prompt after setting the env var: signtype=mock
Then do a rebuild within VS and run tests
If that works, we should update the contrib doc.
It has to do with "public signing" for assembly strong names and the fact that vstest.console defaults to shadow copying, which breaks for public signed assemblies.
Sam Harwell
@sharwell
set SIGNTYPE=mock
devenv
Still no success after a rebuild all
Andrew Arnott
@AArnott
hmmmm... do the tests run from the command line? Can you take a look at the appveyor.yml file to see if it's doing anything that you're not?
bluetarpmedia
@bluetarpmedia
Anyone still use this room or is it archived? :)