Skip to content

Conversation

JasonCard
Copy link
Contributor

No description provided.

wazzamatazz and others added 30 commits April 24, 2020 16:07
Add note to Dispose documentation to highlight that the `CancellationTokenSource` does not cancel its underlying `CancellationToken` when it is disposed, and that cancellation must be triggered separately.
* Updates to code owners

Based on the latest updates to https://github.com/dotnet/runtime/blob/master/docs/area-owners.md

* Update CODEOWNERS
* Auto port System.Threading ACL comments

* Apply suggestions from code review

Co-Authored-By: Koundinya Veluri <[email protected]>

* Apply suggestions from code review

Co-Authored-By: Koundinya Veluri <[email protected]>

Co-authored-by: Koundinya Veluri <[email protected]>
* enable_try_dotnet_to_batch_5b

* removed_extra_line

* Update

* Update

* modified:

* code_review_suggestions_applied

Co-authored-by: Meenal Patel (CSI Interfusion Inc) <[email protected]>
* Replace legacy Code Gallery link with new Samples browser

* Corrected link, using aka.ms to encapsulate query string.

* Fix & usage
* enable_try_dotnet

* updated_last_two_url
* Add missing docs for System.Collections[.Generic]

* Add exceptions

* Add System. prefix to exceptions

* Address feedback - use tool to port exceptions
* Update ObjectSecurity.xml

When you call the method SetAccessRuleProtection with both parameters on true, you may create a non-canonical DACL.
I just made the most important changes. if you want to provide more details and e.g. code to "repair" the ACE order, I think this should go to a separate topic.
I have a colleague who provided sample code.

* Update xml/System.Security.AccessControl/ObjectSecurity.xml

Co-Authored-By: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Security.AccessControl/ObjectSecurity.xml

Co-authored-by: Jeremy Barton <[email protected]>
Co-authored-by: Carlos Sanchez Lopez <[email protected]>
* Add undocumented APIs for S.Diagnostics.Process

* Update xml/System.Diagnostics/Process.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/Process.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessThreadCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* Update xml/System.Diagnostics/ProcessModuleCollection.xml

Co-authored-by: Carlos Sanchez Lopez <[email protected]>

* add more docs

Co-authored-by: Carlos Sanchez Lopez <[email protected]>
"Dataflow block*s* capture the state of the options at their construction."
* add cancelasync note

* standardize notes; remove 'the' .net framework

* add reco for httpclient
* replace linq queryable provider links

Co-authored-by: Genevieve Warren <[email protected]>
* enable_try_dotnet_to_batch_7a

* Update xml/System.Collections.Generic/ICollection`1.xml

Co-authored-by: Maira Wenzel <[email protected]>

* reverting_syntax

* missing_interactive_2208_updated

* Update xml/System.Collections.Generic/Dictionary`2.xml

Co-authored-by: Maira Wenzel <[email protected]>

Co-authored-by: Maira Wenzel <[email protected]>
The way of how Task.Run avoided indenting it's content and hiding the closing braces on the last Console.WriteLine line made it really hard to see what's going on here.
carlossanlop and others added 26 commits July 20, 2020 19:05
Should be zero instead of negative.
Removing System.Windows assembly for System.Windows.Resources.StreamResourceInfo
Changes hecxadecimal to hexadecimal.
* Update ArrayPool`1.xml
Co-authored-by: Stephen Toub <[email protected]>
* MetadataReader: add remarks and example

* Bump .NET Core version to support C# 8.0

Co-authored-by: Andy De George <[email protected]>
* Update DateTime.xml
* Add DispatcherTimer to the list of available timer classes
Typo "TextReader" should read "TextWriter"
* Link to unsafe-escaping section of how-to doc

* Update xml/System.Text.Encodings.Web/JavaScriptEncoder.xml

Co-authored-by: Youssef Victor <[email protected]>

Co-authored-by: Youssef Victor <[email protected]>
* fix broken links

* further fixes

* fix note format

* one more fix
Comment for the last example incorrectly stated the output of the test routine; it showed '' instead of 'null'
Co-authored-by: VSC-Service-Account <[email protected]>
The Remarks for the overload of `Insert` which takes a `CacheDependency` appears to have been copied and pasted from the overload which does not take that parameter and therefore stated that the object added to the cache would be inserted with "no file or cache dependencies", which of course is incorrect. This edit removes that text.
The example is aiming to show a "non available" navigation service. The code should then behave as expected.
Add a remark that reading `ComboBox.Text` is dependent on whether there is a selected item or not.
Use the same wording as documentation for memmove C API.

Fixes dotnet/runtime#6419 (comment)
@dotnet-bot dotnet-bot added this to the July 2020 milestone Jul 22, 2020
@JasonCard JasonCard closed this Jul 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.