Skip to content

extra: %W and %+ support in time::strftime #9689

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 3, 2013
Merged

Conversation

luisbg
Copy link
Contributor

@luisbg luisbg commented Oct 2, 2013

Plus testing added for %X and %x which were supported but not tested.

Working towards #2350

Plus testing added for %X and %x which were supported but not tested.

Working towards rust-lang#2350
@luisbg
Copy link
Contributor Author

luisbg commented Oct 2, 2013

r? @catamorphism
Last one for strftime, tomorrow or Friday I will work on the FIXMEs of strptime.
:)

bors added a commit that referenced this pull request Oct 3, 2013
Plus testing added for %X and %x which were supported but not tested.

Working towards #2350
@bors bors closed this Oct 3, 2013
@bors bors merged commit 8f6e43e into rust-lang:master Oct 3, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Nov 21, 2022
…1995

fix: support `map_or` for `or_fun_call` lint

fixes rust-lang/rust-clippy#8993

The methods defined in `KNOW_TYPES`, except for `map_or`, accepts only one argument, so the matching `if let [arg] = args` works only for these methods.
This PR adds `rest_arg` argument to `check_general_case` method and handling of cases with two arguments to support `map_or`.

changelog: `or_fun_call` support `map_or`

* add `rest_arg` to pass second argument from `map_or(U, F)`
* extract some procedures into closure
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.

3 participants