openium/resgen.swift
1
0
Fork 0
Code Pull Requests Releases 14 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
Member
Copy Link
Copy Source
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
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
Copy Source

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
Copy Source

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
Copy Source

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 labels
Milestone
No items
No Milestone
Assignees
Clear assignees
a.chabozy (Audrey Chabozy) a.valin (Arthur Valin) a.vallet (Aurélien Vallet) b.lambert (Benjamin Lambert) c.guelle (Clémentine Guelle) c.samson (Clément Samson) e.berlinet (Emilie Berlinet) e.guilbault (Emma Guilbault) g.fernandes (Guillaume Fernandes) h.peytavin (Hugues Peytavin) j.neboit (Jean-Charles Neboit) jenkins (Doc Hudson) k.renella (Kévin Renella) l.adlington (Laetitia Adlington) l.godart (Ludovic Godart) l.perret (Loris Perret) l.riviere (Laura Rivière) l.seguin (Léane Seguin) l.zborowski (Lucas Zborowski) n.chaduc (Nicolas Chaduc) o.coudert (Ophélie COUDERT) o.goutet (Olivier Goutet) p.hurdebourcq (Pierre Hurdebourcq) q.bandera (Quentin Bandera) r.bergoin (Richard Bergoin) r.bossin (Rodolphe Bossin) r.portet (Rémy Portet) redmansible (Hans Heeble) s.mellouk (Solène Mellouk) s.tabaka (Simon Tabaka) t.leduc (Thomas Leduc) t.schmitt (Thibaut Schmitt) Tristan-BART (Tristan BART) watea-azure-pipeline (Access Watea Azure Pipeline to SwiftiumKit)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openium/resgen.swift#11