openium/resgen.swift
1
0
Fork 0
Code Pull Requests Releases 12 Activity

DEVTOOLS-185 Remplacer le json en dur des images resgen #11

Merged
q.bandera merged 1 commits from DEVTOOLS-186/SVG_resgen into master 2024-04-19 17:02:26 +02:00
Conversation 2 Commits 1 Files Changed 4 +97 -56
q.bandera commented 2024-04-19 12:52:50 +02:00
Owner
Copy Link
No description provided.
q.bandera added 1 commit 2024-04-19 12:52:50 +02:00
DEVTOOLS-185 Remplacer le json en dur des images resgen
Some checks failed
gitea-openium/resgen.swift/pipeline/head There was a failure building this commit
Details
gitea-openium/resgen.swift/pipeline/pr-master There was a failure building this commit
Details
c8cfe82109
q.bandera requested review from t.schmitt 2024-04-19 12:53:01 +02:00
t.schmitt requested changes 2024-04-19 14:42:59 +02:00
Dismissed
Sources/ResgenSwift/Images/Model/ImageContent.swift Outdated
@@ -0,0 +12,4 @@
let info: Info
static func == (lhs: ImageContent, rhs: ImageContent) -> Bool {
let lhsImagesCount = lhs.images.count
t.schmitt commented 2024-04-19 14:41:26 +02:00
Owner
Copy Link

Tu peux optimiser les performances:

guard lhs.images.count == rhs.images.count else { return false }
let lhsImages = ...
....
return lhsImages == rhsImages

Pas besoin de check les infos.
Si tu as pas le meme count, tu ne vas pas parcourir ton tableau images c'est toujours ca de gagné.

Tu peux optimiser les performances: ``` guard lhs.images.count == rhs.images.count else { return false } let lhsImages = ... .... return lhsImages == rhsImages ``` Pas besoin de check les infos. Si tu as pas le meme count, tu ne vas pas parcourir ton tableau `images` c'est toujours ca de gagné.
q.bandera marked this conversation as resolved
Sources/ResgenSwift/Images/Model/ImageContent.swift Outdated
@@ -0,0 +30,4 @@
let filename: String
}
struct Info: Codable, Equatable {
t.schmitt commented 2024-04-19 14:38:12 +02:00
Owner
Copy Link

Aprs réflexion, on va changer le nom des 3 structs:

  • ImageContent -> AssetContent
  • Image -> AssetImageDescription
  • Info -> AssetInfo
    Ces noms sont plus représentatifs de la data qu'ils contiennent.
Aprs réflexion, on va changer le nom des 3 structs: - `ImageContent` -> `AssetContent` - `Image` -> `AssetImageDescription` - `Info` -> `AssetInfo` Ces noms sont plus représentatifs de la data qu'ils contiennent.
q.bandera marked this conversation as resolved
Sources/ResgenSwift/Images/Model/ParsedImage.swift Outdated
@@ -71,0 +46,4 @@
let encoder = JSONEncoder()
encoder.outputFormatting = .prettyPrinted
guard let data = try? encoder.encode(imageContent) else { return nil }
t.schmitt commented 2024-04-19 14:42:43 +02:00
Owner
Copy Link

Si la méthode appelante ne le fait pas déjà, dans le else regarde comment j'ai fais pour générer un warning pour Xcode. Avec le warning, le dev saura qu'il y a eu un soucis

Si la méthode appelante ne le fait pas déjà, dans le else regarde comment j'ai fais pour générer un warning pour Xcode. Avec le warning, le dev saura qu'il y a eu un soucis
q.bandera added 1 commit 2024-04-19 15:25:00 +02:00
Prise en compte des premiers retorus 636584ca45
t.schmitt approved these changes 2024-04-19 16:48:35 +02:00
q.bandera force-pushed DEVTOOLS-186/SVG_resgen from 636584ca45 to 129eb135f1 2024-04-19 17:02:08 +02:00 Compare
q.bandera merged commit 76ef0a2d59 into master 2024-04-19 17:02:26 +02:00
q.bandera deleted branch DEVTOOLS-186/SVG_resgen 2024-04-19 17:02:26 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
t.schmitt
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
a.chabozy a.valin a.vallet b.lambert c.guelle c.samson c.villeneuve e.berlinet e.guilbault e.joly g.fernandes h.peytavin j.neboit jenkins k.renella l.adlington l.godart l.legrand l.perret l.riviere l.seguin l.zborowski n.chaduc o.coudert o.goutet p.hurdebourcq q.bandera r.bergoin r.bossin r.portet redmansible s.mellouk s.tabaka t.coulange t.leduc t.schmitt watea-azure-pipeline
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openium/resgen.swift#11
Delete Branch "DEVTOOLS-186/SVG_resgen"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?