-  [WT]  [PS]  [Home] [Manage]

[Return]
Posting mode: Reply
  1.   (reply to 4625)
  2. (for post and file deletion)
/pr/ - Programming
  • Supported file types are: C, CSS, DOC, DOCX, GIF, H, JAVA, JPG, PDF, PNG, SVG, SWF, TXT, WEBM
  • Maximum file size allowed is 10000 KB.
  • Images greater than 200x200 pixels will be thumbnailed.
  • Currently 389 unique user posts. View catalog

  • Blotter updated: 2011-01-12 Show/Hide Show All

There's a new /777/ up, it's /gardening/ Check it out. Suggest new /777/s here.

Movies & TV 24/7 via Channel7: Web Player, .m3u file. Music via Radio7: Web Player, .m3u file.

WebM is now available sitewide! Please check this thread for more info.

How bad is it? rayman 14/12/02(Tue)01:39 No. 4625
4625

File iwlmem.c - (1.27KB )

How bad is this piece of code?


>>
Neckbearded Basement Dweller 14/12/03(Wed)11:24 No. 4626

I do not like you inconsistant tab spacing for the first level of nesting....it's....it's driving me nuts... D:
Probably not what you care about, but.... aaaaaah aaaaaaaaaah!


>>
Neckbearded Basement Dweller 14/12/03(Wed)11:30 No. 4627

I have no programs installed to open the file so that must mean it doesn't work right. Also when I open it in Notepad I just see gibberish and some words.


>>
Neckbearded Basement Dweller 14/12/04(Thu)02:32 No. 4629

>>4627
The file works fine. Use a real text editor. If you don't have one you shouldn't be on /pr/.
The reason it displayed wrong for you is it uses line feed for new lines, which is the standard on Linux. Mac uses carriage return and Windows uses both because the deal Bill Gates has with Lucifer says his company must do absolutely everything in the most retarded way possible.


>>
Neckbearded Basement Dweller 14/12/07(Sun)04:40 No. 4633

Not too bad I guess, but you should pretty-format your printf, that's what printf is for after all. do %5x instead of '%x '.
Something else, don't call a char 'ptr', it's misleading, the convention is char *ptr


>>
Neckbearded Basement Dweller 14/12/07(Sun)20:21 No. 4635

>>4629
windows line endings are a pain in the ass, but when you think about it they do kind of make sense, one to go down a line, another to reset to the first position


>>
Neckbearded Basement Dweller 14/12/08(Mon)18:27 No. 4636

This is an unsafe conversion:
>int addr = &ptr;
What if sizeof(int) < sizeof(&ptr)?
Use intptr_t instead.


>>
Neckbearded Basement Dweller 14/12/11(Thu)08:15 No. 4643

>>4625
fucking terrible


>>
legit 14/12/20(Sat)23:34 No. 4648
4648

File iwlmem2.c - (1.19KB )

>>4625

I'm attaching an improved version of your code, if you want more info let me know, I had typed then inside the form, but when I submitted, the site said that the file name already existed and I'm not gonna write it all again if it's not worth it.

BTW, here is your source


>>
Neckbearded Basement Dweller 15/05/22(Fri)20:20 No. 4733

>>4648
>args[0]+2
Isn't that just args[2]?


>>
Nattajerk 15/05/25(Mon)14:47 No. 4736

>>4733
no, that is 2 plus the value of args[0].

args+2 would give you the equivalent of args[0+2].



[Return] [Entire Thread] [Last 50 posts]


Delete post []
Password  
Report post
Reason