[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #22 from Shivansh --- I wont be able to work on this patch as I am facing problem in passing make check, which is taking a lot of time. However, If any changes are required in the current patch

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #21 from Shivansh --- Sorry for the absence. I need to ask a few things: 1) is the key in common/Crypto.cpp in UTC time or localtime? 2) if it is, will it be ok to use already existing method Util::iso8601ToTimestamp() to

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #20 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/commit/ef90709ad1cb6c8cbb649425d0f0c539bbd5f83f tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #19 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/commit/dd014e7029628dd95d2026bcb4fe4a66d75785aa tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #18 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/+/8aea22a32b28728033c4f06a79f93e63b1322151%5E%21 tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #17 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/+/22f1656e08d1cc873a523bf1f869bdd75066dd2a%5E%21 tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #16 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/+/7850c944e204c3a7d62029b2b319eb836c4c06d3%5E%21 tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 Commit Notification changed: What|Removed |Added Whiteboard|target:6.3.0|target:6.3.0 target:6.4.0

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #15 from Shivansh --- Created attachment 151676 --> https://bugs.documentfoundation.org/attachment.cgi?id=151676=edit Poco::Timestamp replacement in Storage.hpp and Storage.cpp Yes, I gave another look to

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #14 from Michael Meeks --- You googled for high resolution std::chrono ? certainly it can represent high resolution time. -- You are receiving this mail because: You are the assignee for the

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #13 from Shivansh --- I am unable to create timestamp of the format Poco::DateTimeFormat::ISO8601_FRAC_FORMAT using std::chrono, due to the decimal in the seconds. Do have any suggestions for this? Or is it okay to go without

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #12 from Michael Meeks --- Create a helper for that that returns a std::chrono in our file utilities - it will be a wrapper of: $ man 2 stat You'll want a#ifdef __APPLE__ to use: st.st_mtimespec there - and should

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #11 from Shivansh --- I have made changes in the declaration part in Storage.hpp(changed Poco::Timestamp to std::time_t). In Storage.cpp, I couldn't find an alternate for the file.getLastModified() in line 266. We might have

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #10 from Michael Meeks --- So - we need to replace the Storage.hpp "FileInfo" class: class FileInfo { public: FileInfo(const std::string& filename, const std::string& ownerId,

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #9 from Shivansh --- For the next change, could you help me to replace this https://opengrok.libreoffice.org/xref/online/wsd/Storage.cpp#266 Thank you. -- You are receiving this mail because: You are the assignee for the

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #8 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/+/8e34705fe22f7efaf44619cf40685217e2216d4b%5E%21 tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #7 from Michael Meeks --- Good question - it would be nice to have a helper function to handle dates that can do that in a Util class and that returns a helpful type for that replacement - perhaps "std::string

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 Commit Notification changed: What|Removed |Added Whiteboard||target:6.3.0 -- You are

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #6 from Commit Notification --- DarkByt31 committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/online/+/a243fdef88e85ec082ade9622374d20fc5a77ffb%5E%21 tdf#107038

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #5 from Shivansh --- I have pushed the commit and sent the license statement. For my next change, in wsd/FileServer.cpp and wsd/LOOLWSD.cpp, I am thinking of replacing `Poco::DateTimeFormatter::format(Poco::Timestamp(),

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #4 from Michael Meeks --- Looks great to me - thanks Shivansh =) can you push to gerrit ? [ and also send a License statement to the mailing list if you havn't already ] cf.

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 Shivansh changed: What|Removed |Added CC||avihs...@gmail.com --- Comment #3

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #2 from Michael Meeks --- Good to do the conversion in small chunks; the easy pieces first and do them very carefully. It would be good to have a number of smaller commits tackling a particular member variable or type.

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 Abhyudaya Sharma changed: What|Removed |Added Assignee|libreoffice-b...@lists.free |sharmaabhyud...@gmail.com

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 Abhyudaya Sharma changed: What|Removed |Added Assignee|sharmaabhyud...@gmail.com |libreoffice-b...@lists.free

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 Michael Meeks changed: What|Removed |Added Keywords|

[Libreoffice-bugs] [Bug 107038] online: Poco::Time* replacement with std::chrono

https://bugs.documentfoundation.org/show_bug.cgi?id=107038 --- Comment #1 from Michael Meeks --- *** Bug 107037 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the