8000 WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime by matrixise · Pull Request #5578 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime #5578

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.

Sign up for GitHub

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

Closed
wants to merge 16 commits into from
Closed

WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime #5578

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
ad5ffab
bpo-1100942: Add datetime.time.strptime and datetime.date.strptime
matrixise Feb 7, 2018
50606eb
Follow the PEP7 for the C Code
matrixise May 15, 2018
ab5030f
Remove unused formats
matrixise May 15, 2018
07fb326
Use subtest for the tests
matrixise Oct 17, 2018
010ac8b
Fix the signatures
matrixise Oct 25, 2018
5b58fc1
Update with the recommendation of @pganssle
matrixise Oct 25, 2018
8353a65
Update
matrixise Nov 4, 2018
cff74a9
Add use case where we generate an exception
matrixise Nov 11, 2018
f92a799
Add the Py_UNUSED(ignore) for datetime_gettime & datetime_getdate
matrixise Nov 11, 2018
4c4a7cc
Add a link to the strtime() and strptime() behaviour
matrixise Mar 6, 2019
b006172
Update the blurb entry and add an entry in the Doc/whatsnew/3.8.rst
matrixise Mar 6, 2019
ff7487f
Fix doc
matrixise Mar 6, 2019
a048f0e
Convert the _check_invalid_datetime_spec to the C-API
matrixise Mar 18, 2019
9ac0af4
Support tuple and list for the specs
matrixise Mar 18, 2019
a952bb7
_datetimemodule._check_invalid_datetime_specs only accepts tuple and not
matrixise Mar 24, 2019
412aea2
Fix the code with the recommendation of Martin Panter
matrixise Mar 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Support tuple and list for the specs
  • Loading branch information
@matrixise
matrixise committed Mar 22, 2019
commit 9ac0af48a1cc7504d876c78939475360efba3d5d
9 changes: 7 additions & 2 deletions Modules/_datetimemodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6360,13 +6360,18 @@ _check_invalid_datetime_specs(PyObject *module, PyObject *args)
return NULL;
}

assert(PyTuple_CheckExact(specs) || PyList_CheckExact(specs));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would it be a list? This seems like unused code.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the first version was only for the tuples. After, I wanted to accept the list for my tests. but I am not against to only accept the tuples.


PyObject *found_invalid_specs = PyList_New(0);
if (found_invalid_specs == NULL) {
return NULL;
}

for (Py_ssize_t pos = 0; pos < PyTuple_GET_SIZE(specs); pos++) {
PyObject *spec = PyTuple_GET_ITEM(specs, pos);
PyObject *(*GetItem)(PyObject *, Py_ssize_t) = \
PyTuple_CheckExact(specs) ? PyTuple_GetItem : PyList_GetItem;

for (Py_ssize_t pos = 0; pos < PyObject_Size(specs); pos++) {
PyObject *spec = GetItem(specs, pos);
if (PySequence_Contains(format, spec)) {
PyList_Append(found_invalid_specs, spec);
}
Expand Down
0