profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/wongzigii/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Zigii Wong wongzigii Shanghai, China Think different

SwiftyJSON/SwiftyJSON 20856

The better way to deal with JSON data in Swift.

wongzigii/Butterfly 413

:airplane: A tiny library for integrating feedback feature. Compatible with Swift 4.0.

rrbe/egg-passport-qq 4

egg-passport plugin for qq-oauth-login

wongzigii/1-liners 0

Functional tools that couldn’t be simpler.

wongzigii/3030 0

3030 game: Don't touch the walls! (Thanks, Chrome dev team!)

wongzigii/Action 0

Abstracts actions to be performed in RxSwift.

wongzigii/Alamofire 0

Elegant HTTP Networking in Swift

wongzigii/Alamofire-SwiftyJSON 0

Alamofire extension for serialize NSData to SwiftyJSON

wongzigii/Alcatraz 0

Package manager for Xcode

wongzigii/alfred-homebrew 0

Easily control Homebrew with Alfred

create barnchXiaoTeTech/lottie-ios

branch : xiaote

created branch time in 25 days

issue commentyedf/dtm

有偿招募英文文档维护者

报名

yedf2

comment created time in a month

startedcoteditor/CotEditor

started time in a month

startedzenustech/zeno

started time in a month

issue openedapollographql/apollo-ios

Upload file with progress

How to upload file with progress? The upload function in ApolloClient doesn't have a progress closure.

created time in a month

Pull request review commentapple/swift

Fixes SR-14951 : Print message of @available attribute within Protocol

 NOTE(declared_protocol_conformance_here,none,      (Type, unsigned, Identifier, Identifier))  ERROR(witness_unavailable,none,-        "unavailable %0 %1 was used to satisfy a requirement of protocol %2",-        (DescriptiveDeclKind, DeclName, Identifier))+        "unavailable %0 %1 was used to satisfy a requirement of protocol %2%select{|: %3}3",

@RAJAGOPALAN-GANGADHARAN Hello, I am new to Diagnostics system, what's the meaning of |: in this case?

RAJAGOPALAN-GANGADHARAN

comment created time in 2 months

PullRequestReviewEvent

delete branch wongzigii/swift-1

delete branch : SR-14886

delete time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

Squashed into a single commit. Done.

wongzigii

comment created time in 2 months

push eventwongzigii/swift-1

ZigiiWong

commit sha 26b3c55c230c5569aa98bfdcdb27f375aa414af4

[SR-14886] Check wCSIA before copying byte by byte.

view details

push time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

Ah, Nice! I was thinking about 1..<2 is an array, but actually it is an Range. :D Updated again and would be good to go now.

wongzigii

comment created time in 2 months

push eventwongzigii/swift-1

ZigiiWong

commit sha 1687f3b54a2d2e624f6c7536eb5e817a78070cc5

Add test case for copyBytes without contiguouseStorage

view details

push time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

Updated test case!

wongzigii

comment created time in 2 months

push eventwongzigii/swift-1

ZigiiWong

commit sha cd997fc4825e6007a9432a9e6a51a294a0d0bbcb

Add test case for copyBytes without contiguouseStorage

view details

push time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

Just to clarify, should I need to use a separately memory layout array to test the old path?

From https://forums.swift.org/t/contiguous-collection-protocols/17875/6, a separately array would need be nested in a class with NSArray bridging.

wongzigii

comment created time in 2 months

issue closedaws-amplify/amplify-cli

amplify update auth failed

Before opening, please confirm:

  • [X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
  • [X] I have searched for duplicate or closed issues.
  • [X] I have read the guide for submitting bug reports.
  • [X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?

npm

If applicable, what version of Node.js are you using?

v14.16.0

Amplify CLI Version

5.1.2

What operating system are you using?

Mac

Amplify Categories

auth

Amplify Commands

update

Describe the bug

TypeError: Cannot read property 'substring' of undefined

amplify update auth
Please note that certain attributes may not be overwritten if you choose to use defaults settings.

You have configured resources that might depend on this Cognito resource.  Updating this Cognito resource could have unintended side effects.

Using service: Cognito, provided by: awscloudformation
 What do you want to do? Update OAuth social providers
 Select the identity providers you want to configure for your user pool:
TypeError: Cannot read property 'substring' of undefined
    at userPoolDefaults (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/assets/cognito-defaults.js:24:47)
    at Object.identityAndUserPoolDefaults [as identityPoolAndUserPool] (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/assets/cognito-defaults.js:66:6)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/utils/auth-defaults-appliers.ts:46:54
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/handlers/resource-handlers.ts:45:31
    at Object.executeAmplifyCommand (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/index.js:379:3)
    at executePluginModuleCommand (/usr/local/lib/node_modules/@aws-amplify/cli/src/execution-manager.ts:177:3)
    at Object.executeCommand (/usr/local/lib/node_modules/@aws-amplify/cli/src/execution-manager.ts:29:5)
    at Object.run (/usr/local/lib/node_modules/@aws-amplify/cli/src/index.ts:151:5)
There was an error adding the auth resource

Expected behavior

Command execute success without error

Reproduction steps

1.Run amplify update auth 2.Update OAuth social providers 3.Select Google

GraphQL schema(s)

<details>

# Put schemas below this line


</details>

Log output

<details>

# Put your logs below this line


</details>

Additional information

No response

closed time in 2 months

wongzigii

issue commentaws-amplify/amplify-cli

amplify update auth failed

OK Good to know Thanks @SwaySway Closing

wongzigii

comment created time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

Is it true that RawBufferCopyBytes is 5694x faster than before? Can't believe that.

wongzigii

comment created time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

Can somebody trigger the @swift-ci again ?

wongzigii

comment created time in 2 months

issue commentaws-amplify/amplify-cli

amplify update auth failed

I've renamed the project directory name, maybe that's the issue.

wongzigii

comment created time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

@glessard Done. If buffer.baseAddress == nil, then buffer.count would be zero.

wongzigii

comment created time in 2 months

push eventwongzigii/swift-1

Andrew Trick

commit sha 851bfeb3188a956f96e464c8b16441521f38cfa2

Fix copy_value to have 'None' side-effects. Copies can be moved as much as you like as long as OSSA is legal. This fixes some instruction deletion utilities for OSSA and any other utilities that check side effects. Copies are common. It also finally allows pure functions to be CSE'd!

view details

Andrew Trick

commit sha 988ff18f0697cf752c7e3d2b537eb68ac8416794

Merge branch 'main' into fix-copy-sideeffect

view details

Alex Hoppen

commit sha 7525ebf21c028f330f3be285e8c230f6de4cac93

[CodeCompletion] Add test case for SR-14711 The issue has been fixed on `main` recently. Since we have a reproducing test case, add it to the test suite.

view details

Alex Hoppen

commit sha 2eeff365b1e2190e6c2b5c7f45196aa379e788a7

[CodeCompletion] Record key path component types in the constraint system solution The added test case fails because the result builder inside `List2` is being type checked twice: Once for every overload of `List2`. Because of the way that result builders are being type checked right now, each overload of `List2` creates a new type variable for the key components in `foo` and the constraint system only keeps track of the key path component -> type mapping for the last solution discovered. When we are now trying to look up the type of the first key path component for the first solution, the constraint system returns the type variable that is being used by the second solution and simplifying that type variable through the first solution fails because it doesn’t know about the type variable. To fix the issue, make sure that the solutions keep track of the their type variables associated to key path components. That way the first solution owns its own key path component -> type variable mapping and is thus also able to simplify the type variable it has associated with the component. Fixes rdar://80522345 [SR-14916]

view details

ZigiiWong

commit sha efb063907546a8643fbfe0024fae01ea9325eb1e

Improve diagnostic error message for empty cases

view details

Victoria Mitchell

commit sha f53a135dca5065c34025e5d2ecd36d668fc220b1

don't skip walking VarDecls from parent clang nodes rdar://80235766

view details

Saleem Abdulrasool

commit sha f7bddc4557259cf1360324b429fce8c165213f24

update_checkout: add a `--skip-tags` option When fetching the sources, the tags take a very long time due to the daily tags. Add an option to skip fetching tags.

view details

Victoria Mitchell

commit sha 6887212a55a0696dfea85388b4a22c5502ca0e37

check for parent pattern bindings when the parent is a decl

view details

Eric Miotto

commit sha 66bd9f80b4286f9801283c8dc35af3f42a432975

[build] explicitly set OSX_ARCHITECTURES for targets in stdlib Our custom build code already sets the architecture of compilation commands by different means, and so far we were relying on setting `CMAKE_OSX_ARCHITECTURES` to an empty value to avoid CMake doing the same. However, on Apple Silicon, CMake 3.19+ enforces a default value in this scenario (https://gitlab.kitware.com/cmake/cmake/-/merge_requests/5291/diffs), and this would result in the inability to compile code for x86_64. Addresses SR-14035, rdar://80613594

view details

Rintaro Ishizaki

commit sha b455177f9e1da5af8a68403ad4e8eb51807e7662

[Syntax] Make `PostfixIfConfigExpr.base` optional For nested `#if ... #endf` in postfix if-config expression, like: baseExpr #if COND1 #if COND2 .member #endif #endif Consider the inner `#if` be a postfix if-config with 'nil' base expression. https://bugs.swift.org/browse/SR-14929

view details

Kavon Farvardin

commit sha 5ffe8e08788c5cb5df609ded04dbcc381cd9ce4a

fix parsing of `isolated` as an argument label With the introduction of `isolated` as a type modifier for actor types, the parsing of a parameter regressed such that `isolated` was no longer accepted as an ordinary argument label. This patch fixes that and adds a little lookahead utility to clean-up the code that disambiguates the uses of `isolated` as either a label or a type modifier. Resolves rdar://80300022

view details

Guillaume Lessard

commit sha a5feb1ab6b25fc578b16ffeebb6be6030ac1a430

detect failed allocations from `posix_memalign` - `posix_memalign` doesn't touch its pointer-to-pointer parameter when it fails, so `AlignedAlloc` ended up returning stack garbage in that case.

view details

Guillaume Lessard

commit sha 7f68d6a4f5fabe02fdd59e66135c6eaa4f1cc356

document the requirement that allocation alignments must be powers of two.

view details

Ben Barham

commit sha f9b3e40595439a9107bb9de4f70e7cae8d2aa945

[SourceKit/DocInfo] Add is_async for properties with async getters Resolves rdar://80546521

view details

Varun Gandhi

commit sha fa406759108100b8111f53155e9e04f433de8997

Further workaround for priority mishandling in runtime. See rdar://79378762, SR-14802, SR-14841, SR-14875. This doesn't resolve all hangs, such as those occurring due to explicit usage of priorities and certain other situations where priorities seem to be causing issues (rdar://79823345), but it does resolve some cases.

view details

Alastair Houghton

commit sha 895bf266ddcdec1b72829d7647b97cdec88a7834

Fix tab size for Emacs users. For some reason .dir-locals.el was setting tab-width to 2. That's a Really Bad Idea, because if somehow (as happened in my case) indent-tabs-mode was 't instead of nil like it should have been, Emacs will insert tabs instead of spaces, and it'll *look* like it's doing the right thing.

view details

Varun Gandhi

commit sha 968ef77fae7077d55be7b1906266a0ef504afd1d

Merge pull request #38391 from wongzigii/SR-14883 [SR-14883] Improve diagnostic error message for empty cases

view details

Rintaro Ishizaki

commit sha 0441183f462568f5971f86c0f55a86576f09a07a

Merge pull request #38410 from rintaro/syntax-postfixifconfig-sr14929 [Syntax] Make `PostfixIfConfigExpr.base` optional

view details

Guillaume Lessard

commit sha e946cd02c048a0c55adbc24a314c6443e0121e5d

remove duplicated code - call `UnsafeMutablePointer.allocate(capacity:)` from `UnsafeMutableBufferPointer.allocate(capacity:)`. - This was a straight code duplication.

view details

Guillaume Lessard

commit sha ae10c297a97f36de6f6b59aa08f0836aaafbaad5

make documentation of UMRBP.allocate() more complete.

view details

push time in 2 months

issue openedaws-amplify/amplify-cli

amplify update auth

Before opening, please confirm:

  • [X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
  • [X] I have searched for duplicate or closed issues.
  • [X] I have read the guide for submitting bug reports.
  • [X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?

npm

If applicable, what version of Node.js are you using?

v14.16.0

Amplify CLI Version

5.1.2

What operating system are you using?

Mac

Amplify Categories

auth

Amplify Commands

update

Describe the bug

amplify update auth Please note that certain attributes may not be overwritten if you choose to use defaults settings.

You have configured resources that might depend on this Cognito resource. Updating this Cognito resource could have unintended side effects.

Using service: Cognito, provided by: awscloudformation What do you want to do? Update OAuth social providers Select the identity providers you want to configure for your user pool: TypeError: Cannot read property 'substring' of undefined at userPoolDefaults (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/assets/cognito-defaults.js:24:47) at Object.identityAndUserPoolDefaults [as identityPoolAndUserPool] (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/assets/cognito-defaults.js:66:6) at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/utils/auth-defaults-appliers.ts:46:54 at processTicksAndRejections (internal/process/task_queues.js:93:5) at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/provider-utils/awscloudformation/handlers/resource-handlers.ts:45:31 at Object.executeAmplifyCommand (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-auth/src/index.js:379:3) at executePluginModuleCommand (/usr/local/lib/node_modules/@aws-amplify/cli/src/execution-manager.ts:177:3) at Object.executeCommand (/usr/local/lib/node_modules/@aws-amplify/cli/src/execution-manager.ts:29:5) at Object.run (/usr/local/lib/node_modules/@aws-amplify/cli/src/index.ts:151:5) There was an error adding the auth resource

Expected behavior

Command execute success without error

Reproduction steps

Just Run amplify update auth

GraphQL schema(s)

<details>

# Put schemas below this line


</details>

Log output

<details>

# Put your logs below this line


</details>

Additional information

No response

created time in 2 months

startedpowertab/powertabeditor

started time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

@weissi

wongzigii

comment created time in 2 months

pull request commentapple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

@swift-ci please smoke test

wongzigii

comment created time in 2 months

PR opened apple/swift

Check withContiguousStorageIfAvailable before copying byte by byte

<!-- What's in this pull request? --> Check withContiguousStorageIfAvailable before copying byte by byte.

<!-- If this pull request resolves any bugs in the Swift bug tracker, provide a link: --> Resolves SR-14886.

<!-- Before merging this pull request, you must run the Swift continuous integration tests. For information about triggering CI builds via @swift-ci, see: https://github.com/apple/swift/blob/master/docs/ContinuousIntegration.md#swift-ci

Thank you for your contribution to Swift! -->

+8 -0

0 comment

1 changed file

pr created time in 2 months

create barnchwongzigii/swift-1

branch : SR-14886

created branch time in 2 months

created tagwongzigii/swift-1

tagswift-5.5-DEVELOPMENT-SNAPSHOT-2021-07-16-a

The Swift Programming Language

created time in 2 months